Re: Policycoreutils latest diffs.

[Ivan Gyurdiev <ivg2@cornell.edu>]

> -			rc = commands.getstatusoutput("grep -v '^ *#' %s" % self.selinuxdir+self.type+"/seusers")
> -			if rc[0] == 0 and rc[1] != "":
> -				ulist = rc[1].split("\n")
> -				for u in ulist:
> -					if len(u)==0:
> +			try:
> +				fd =open(self.selinuxdir+self.type+"/seusers")
> +				for u in  fd.read().split('\n'):
> +					u=u.strip()
> +					if len(u)==0 or u[0]=="#":
>  						continue
>  					user = u.split(":")
>  					if len(user) < 3:
>  						continue
>   
Somebody pointed out that len(user) < 3 assumes the mls component of the 
seusers file is required.
This is not consistent with libsemanage and libselinux, which consider 
it optional.


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.