<div>&nbsp;</div>
<div><strong>Conditions</strong>:</div>
<div>
<p>In /openbsm/libbsm/bsm_flags.c, function getauditflagschar() converts the au_mask_t fields into a string value. There are codes followed as:</p>
<p><font face="courier new,monospace">1:&nbsp;for (setauclass(); (c = getauclassent()) != NULL;<br>2:&nbsp;&nbsp;&nbsp;&nbsp; free_au_class_ent(c)) {<br>&nbsp;&nbsp;&nbsp;&nbsp; <br>3:&nbsp;&nbsp;&nbsp;&nbsp; ... ...<br>&nbsp;&nbsp;&nbsp;&nbsp; <br>4:&nbsp;}</font></p>
<p>In /openbsm/libbsm/bsm_flags.c, function getauclassent() returns the next au_class_ent structure from the file. There are codes followed as:&nbsp;&nbsp;&nbsp;&nbsp; </p>
<p><font face="courier new,monospace">&nbsp;5: /* Parse tokptr to au_class_ent components. */<br>&nbsp;6: if (classfromstr(tokptr, delim, c) == NULL) {<br>&nbsp;7:&nbsp; ... ...<br>&nbsp;8:&nbsp;&nbsp; return (NULL);<br>&nbsp;9: }</font></p>
<p><strong>If </strong></p>
<p>(1) there are lines in /etc/security/audit_class :</p>
<p><font face="courier new,monospace">#<br># This file must match audit.h<br>#<br>0x00000080:pc:process</font></p>
<p>(2)&nbsp; in my own program, there are</p>
<p><font face="courier new,monospace">&nbsp;&nbsp;&nbsp; char flags[256];<br>&nbsp;&nbsp;&nbsp; au_mask_t pmask;<br>&nbsp;&nbsp;&nbsp; pmask.am_failure=pmask.am_failure = 128;<br>&nbsp;&nbsp;&nbsp; (void) getauditflagschar(flags, &amp;pmask, 0);<br>&nbsp;&nbsp;&nbsp; printf(&quot;flags=%s&quot;,flags);
</font><br></p>
<p>The wantted&nbsp;result should be:<br><font face="courier new,monospace">&nbsp; flags=pc</font><br><strong></strong></p>
<p><strong>BUG</strong>:</p>
<p>The result now is:<br><font face="courier new,monospace">&nbsp; flags=</font></p>
<p><strong>Reason</strong>:</p>
<p>The function getauclassent()&nbsp; will always&nbsp;return NULL in line 8, when reading &quot;#&quot; from configure file /etc/security/audit_class. So the function getauditflagschar()&nbsp; will always return in line 1. </p>
<p>In my opinion, the&nbsp;for{} statement should loop&nbsp;until file is ended. &nbsp;</p>
<p>Any problems in my opinion? And how to patch it?</p></div>