antirez / antirez/sds

Memleak in sds.c:40

Offen
#87 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C
Sterne
5.6k
Forks
510
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

There's a possible memleak and NULL ptr usage in sds.c:40 in `sdssplitargs`:

```c
vector = s_realloc(vector,((*argc)+1)*sizeof(char*));
vector[*argc] = current;
```

Realloc here might return NULL and thus NULLify `vector` while losing the reference to original object. I believe something like the following should be done:
```c
char** new_vector = s_realloc(vector,((*argc)+1)*sizeof(char*));
if (new_vector == NULL) goto err;
vector = new_vector;
vector[*argc] = current;
```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.