On 02/14/13 17:42, John Ferlan wrote:
Coverity found the DACGenLabel was checking for mgr == NULL after a
possible dereference; however, in order to get into the function the
virSecurityManagerGenLabel would have already dereferenced sec_managers[i]
so the check was unnecessary. Same check is made in SELinuxGenSecurityLabel.
---
src/security/security_dac.c | 6 ------
src/security/security_selinux.c | 6 ------
2 files changed, 12 deletions(-)
Yep, probably a relic since layered security drivers were added.
ACK.
Peter