On Tue, Jun 14, 2016 at 1:19 PM, Casey Schaufler <[email protected]> wrote: > On 6/14/2016 11:57 AM, Kees Cook wrote:
>> it's okay.) Also, should lsm == NULL be checked early and >> rejected/skipped so the lsm != NULL test isn't needed in both loops? > > Ah, you miss the nuance of the code. NULL is an acceptance condition, > not a rejection as it would be in most cases. Ah! Yes, got it now. Thanks! -Kees -- Kees Cook Chrome OS & Brillo Security

