redis / redis

Redis is an in-memory database that persists on disk. The data model is key-value, but many different kind of values are supported: Strings, Lists, Sets, Sorted Sets, Hashes, Streams, HyperLogLogs, Bitmaps.

Home Page:http://redis.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[BUG] CWE: 664

Ionescu-Ioan opened this issue · comments

Description of the bug:

After I used Cppcheck to perform static analysis of the source code of Redis I discoverd the following error:
CWE 664 : "va_list ‘_cpy’ used before va_start() was called" in deps/hiredis/hiredis.c

I'm not sure yet if it's a false positive or not, but if it's really a bug, it might affect the good functionality of the project.

i think it is a false positive.
the code copies ap to _cpy via va_copy(), so it doesnt need a va_start().