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

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to