antirez / antirez/sds

Memleak in sds.c:40

オープン
#87 コメント 0 件 リアクション 1 件 担当者 0 名 GitHub で見る
主要言語
C
スター
5.6k
フォーク
510
PR マージ指標
30日以内にマージされた PR はありません

説明

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;
```

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。