Changeset 570 for trunk/src/sh_static.c

Timestamp:

Sep 5, 2021, 12:40:51 PM (3 years ago)

Author:

katerina

Message:

Fixes for some compile/cppcheck warnings, version 4.4.6.

File:

• trunk/src/sh_static.c (modified) (4 diffs)

trunk/src/sh_static.c

@@ -464 +464 @@
 {
         FILE *grf;
-        gid_t *group_list;
-        int num_groups, rv;
+        gid_t *group_list = NULL;
+        size_t num_groups;
+        int rv;
         char **m;
         struct group group;
@@ -474 +475 @@
 
         /* We alloc space for 8 gids at a time. */
-        if (((group_list = calloc(8,sizeof(gid_t *))) != NULL)
+        if (buff && ((group_list = calloc(8,sizeof(gid_t *))) != NULL)
                 && ((grf = fopen(_PATH_GROUP, "r")) != NULL)
                 ) {
@@ -489 +490 @@
                                         if (!strcmp(*m, user)) {
                                                 if (!(num_groups & 7)) {
-                                                        gid_t *tmp = (gid_t *)
-                                                                realloc(group_list,
-                                                                                (num_groups+8) * sizeof(gid_t *));
-                                                        if (!tmp) {
-                                                                rv = -1;
-                                                                goto DO_CLOSE;
-                                                        }
-                                                        group_list = tmp;
+                                                  gid_t *tmp = NULL;
+                                                  if (num_groups > (SIZE_MAX - 8)) {
+                                                    rv = -1;
+                                                    goto DO_CLOSE;
+                                                  }
+                                                  if ((num_groups+8) <= (SIZE_MAX / sizeof(gid_t *))) {
+                                                    tmp = (gid_t *)
+                                                      realloc(group_list,
+                                                              (num_groups+8) * sizeof(gid_t *));
+                                                  } 
+                                                  if (!tmp) {
+                                                    rv = -1;
+                                                    goto DO_CLOSE;
+                                                  }
+                                                  group_list = tmp;
                                                 }
                                                 group_list[num_groups++] = group.gr_gid;
@@ -512 +520 @@
         /* group_list will be NULL if initial malloc failed, which may trigger
          * warnings from various malloc debuggers. */
-        free(group_list);
-        free(buff);
+        if (group_list) free(group_list);
+        if (buff) free(buff);
+        /* cppcheck-suppress resourceLeak */
         return rv;
 }