Changeset 1442:915450ee91c7 in freeDiameter for extensions/app_diameap/diameap_server.c

Timestamp:

Feb 27, 2020, 2:19:24 PM (4 years ago)

Author:

Luke Mewburn <luke@mewburn.net>

Branch:

default

Phase:

public

Message:

fix gcc 9.1.1 compile warnings

Use snprintf instead of strncpy to ensure NUL termination.

Simplify attribute copying to calloc/memcpy instead
of malloc/memset/strncpy/assignment.

Appeases gcc 9.1.1 warnings about strncpy bounds.

File:

• extensions/app_diameap/diameap_server.c (modified) (1 diff)

extensions/app_diameap/diameap_server.c

@@ -1961 +1961 @@
         {
         case 1:
-                attribute_op = malloc(strlen(op));
-                memset(attribute_op, 0, strlen(op));
-                strncpy(attribute_op, op + 1, strlen(op) - 1);
-                attribute_op[strlen(op)] = '\0';
+                attribute_op = calloc(strlen(op), sizeof(char *));
+                memcpy(attribute_op, op + 1, strlen(op) - 1);
                 break;
         case 2:
-                attribute_op = malloc(strlen(op));
-                memset(attribute_op, 0, strlen(op));
-                strncpy(attribute_op, op, strlen(op) - 1);
-                attribute_op[strlen(op)] = '\0';
+                attribute_op = calloc(strlen(op), sizeof(char *));
+                memcpy(attribute_op, op, strlen(op) - 1);
                 break;
         default:
-                attribute_op = malloc(strlen(op) + 1);
-                memset(attribute_op, 0, strlen(op) + 1);
-                strcpy(attribute_op, op);
-                attribute_op[strlen(op) + 1] = '\0';
+                attribute_op = calloc(strlen(op) + 1, sizeof(char *));
+                memcpy(attribute_op, op, strlen(op));
         }
         if (strcmp(attribute_op, "=") == 0)