From: Kamalesh Babulal Date: Tue, 31 May 2022 22:51:21 +0000 (-0600) Subject: config: add missing ret value check from getgrnam_r() X-Git-Tag: v3.0~67 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=18cc60ddf3cba088fce3062099d8ee5fca5341a0;p=thirdparty%2Flibcgroup.git config: add missing ret value check from getgrnam_r() Fix Unchecked return values from library, reported by Coverity tool: CID 258287 (#1 of 1): Unchecked return value from library (CHECKED_RETURN). check_return: Calling getgrnam_r(value, group, buffer, 20480UL, &group_buffer) without checking return value. This library function may fail and return an error code. CID 258303 (#1 of 1): Unchecked return value from library (CHECKED_RETURN). check_return: Calling getgrnam_r(value, group, buffer, 20480UL, &group_buffer) without checking return value. This library function may fail and return an error code. Coverity expects us to check for return value from getgrnam_r(), instead of the current for group_buffer != NULL. Which is right, let's make Coverity happy by moving the check to return value. Signed-off-by: Kamalesh Babulal Signed-off-by: Tom Hromatka --- diff --git a/src/config.c b/src/config.c index 56876fb6..3c042da5 100644 --- a/src/config.c +++ b/src/config.c @@ -365,10 +365,8 @@ int config_group_task_perm(char *perm_type, char *value, int flag) if (!group) goto group_task_error; - getgrnam_r(value, group, buffer, - CGROUP_BUFFER_LEN, &group_buffer); - - if (group_buffer == NULL) { + if (getgrnam_r(value, group, buffer, + CGROUP_BUFFER_LEN, &group_buffer) != 0) { free(group); goto group_task_error; } @@ -475,9 +473,8 @@ int config_group_admin_perm(char *perm_type, char *value, int flag) if (!group) goto admin_error; - getgrnam_r(value, group, buffer, - CGROUP_BUFFER_LEN, &group_buffer); - if (group_buffer == NULL) { + if (getgrnam_r(value, group, buffer, + CGROUP_BUFFER_LEN, &group_buffer) != 0) { free(group); goto admin_error; }