Replace unsafe strcpy with strncpy.

Valgrind is happy now.
This commit is contained in:
Chintan Parikh
2017-10-02 16:06:32 -07:00
parent 4b8f7637b1
commit e902089402

View File

@@ -76,10 +76,10 @@ DIALData *retrieve_dial_data(char *app_name) {
char value[256]; char value[256];
while (fscanf(f, "%255s %255s\n", key, value) != EOF) { while (fscanf(f, "%255s %255s\n", key, value) != EOF) {
DIALData *newNode = (DIALData *) malloc(sizeof(DIALData)); DIALData *newNode = (DIALData *) malloc(sizeof(DIALData));
newNode->key = (char *) malloc(strlen(key)); newNode->key = (char *) calloc(1, strlen(key));
strcpy(newNode->key, key); strncpy(newNode->key, key, strlen(key));
newNode->value = (char *) malloc(strlen(value)); newNode->value = (char *) calloc(1, strlen(value));
strcpy(newNode->value, value); strncpy(newNode->value, value, strlen(value));
newNode->next = result; newNode->next = result;
result = newNode; result = newNode;
} }