xen discuss - Jun 2008 - i.minorperm class action script

Trying to test my fix for :

6716457 LU to build 92 doesn''t fix permissions for the xVM devices

I noticed that running the CAS on / still didn''t chown the permissions.
And indeed:

                #  For all entries in minor_perm whose attributes had
                #  to be corrected, correct the relevant attributes of
                #  the
                #  already-existing devices that correspond to those
                #  entries.
                #
                if [ -s /tmp/chdevs.$$ -a "$PKG_INSTALL_ROOT" !=
""  -a \
                     "$PKG_INSTALL_ROOT" != "/" ] ; then

Anyone know why this is true? Also, can somebody suggest a good reviewer
for (or review) the fix:

http://cr.opensolaris.org/~johnlev/leastpriv-lu

thanks
john

John Levon wrote:
> Trying to test my fix for :
> 
> 6716457 LU to build 92 doesn''t fix permissions for the xVM devices
> 
> I noticed that running the CAS on / still didn''t chown the
permissions.
> And indeed:
> 
>                 #  For all entries in minor_perm whose attributes had
>                 #  to be corrected, correct the relevant attributes of
>                 #  the
>                 #  already-existing devices that correspond to those
>                 #  entries.
>                 #
>                 if [ -s /tmp/chdevs.$$ -a "$PKG_INSTALL_ROOT" !=
""  -a \
>                      "$PKG_INSTALL_ROOT" != "/" ] ;
then
> 
> Anyone know why this is true? Also, can somebody suggest a good reviewer
> for (or review) the fix:
> 
> http://cr.opensolaris.org/~johnlev/leastpriv-lu
I''m not too familiar with the handling of device permissions on
upgrade,
but LGTM anyways.

I was able to test this fix by mounting your new SUNWcsd on top of a 
snv_92 iso image, and everything worked after another lu.

-Ryan
> 
> thanks
> john
> _______________________________________________
> xen-discuss mailing list
> xen-discuss@opensolaris.org