On 03/09/2010 05:16 PM, KaiGai Kohei wrote: > (2010/03/10 5:08), Eamon Walsh wrote: > >> What is the meaning of the fourth "temp" argument here. Aren't lines >> only supposed to have three entries? Please remove this if it is not needed. >> > If the line has four or more elements, we want to raise an error, rather than > ignoring it. > If here is not the fourth %as, sscanf() does not return 4, does it? > > Thanks, > OK, that makes sense. I don't see any other issues with the patches. Unless Andy has any concerns, I will push them later today or tomorrow. -- Eamon Walsh National Security Agency -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.