diff options
| author | Rich Megginson <rmeggins@redhat.com> | 2010-07-06 16:47:45 -0600 |
|---|---|---|
| committer | Rich Megginson <rmeggins@redhat.com> | 2010-07-06 17:41:20 -0600 |
| commit | 85eb9216d5d4163139a90432084452cf81c8444e (patch) | |
| tree | a1f1306ce6cb9f2d346eaddefcf75a29d29f91f0 /include | |
| parent | 7a9c069a7d64c1370353278c34bf9065aeb604ea (diff) | |
| download | ds-85eb9216d5d4163139a90432084452cf81c8444e.tar.gz ds-85eb9216d5d4163139a90432084452cf81c8444e.tar.xz ds-85eb9216d5d4163139a90432084452cf81c8444e.zip | |
Bug 611850 - fix coverity Defect Type: Error handling issues
https://bugzilla.redhat.com/show_bug.cgi?id=611850
Resolves: bug 611850
Bug Description: fix coverity Defect Type: Error handling issues
Reviewed by: nhosoi (Thanks!)
Branch: HEAD
Fix Description: Check the error return from the functions. In some cases,
I was able to figure out that the calling function should perform additional
error handling (return early, goto error label), but in general the code
just logs an appropriate error message and continues. I was able to get
rid of some more libacl code. I removed an unused variable from modify.c
Platforms tested: RHEL5 x86_64
Flag Day: no
Doc impact: no
Diffstat (limited to 'include')
| -rw-r--r-- | include/libaccess/aclproto.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/include/libaccess/aclproto.h b/include/libaccess/aclproto.h index 68ef3d80..f1371cfc 100644 --- a/include/libaccess/aclproto.h +++ b/include/libaccess/aclproto.h @@ -151,7 +151,6 @@ NSAPI_PUBLIC int ACL_Init(void); NSAPI_PUBLIC int ACL_InitPostMagnus(void); NSAPI_PUBLIC int ACL_LateInitPostMagnus(void); NSAPI_PUBLIC void ACL_ListHashUpdate(ACLListHandle_t **acllistp); -NSAPI_PUBLIC void ACL_GetPathAcls(char *path, ACLListHandle_t **acllist_p, char *prefix, ACLListHandle_t *masterlist); NSAPI_PUBLIC int ACL_ReadDbMapFile(NSErr_t *errp, const char *map_file, int default_only); NSAPI_PUBLIC int ACL_MethodNamesGet(NSErr_t *errp, char ***names, int *count); |