Fix Honeywell-CM921 VLA use (#1719)
This commit is contained in:
parent
215fd3b567
commit
b07bae3dbb
1 changed files with 2 additions and 2 deletions
|
@ -53,9 +53,9 @@ typedef struct {
|
||||||
static data_t *add_hex_string(data_t *data, const char *name, const uint8_t *buf, size_t buf_sz)
|
static data_t *add_hex_string(data_t *data, const char *name, const uint8_t *buf, size_t buf_sz)
|
||||||
{
|
{
|
||||||
if (buf && buf_sz > 0) {
|
if (buf && buf_sz > 0) {
|
||||||
char tstr[(buf_sz * 2) + 1]; // note: VLA should not be used
|
char tstr[256] = {0};
|
||||||
char *p = tstr;
|
char *p = tstr;
|
||||||
for (unsigned i = 0; i < buf_sz; i++, p+=2)
|
for (unsigned i = 0; (i < buf_sz) && (i < 127); i++, p+=2)
|
||||||
sprintf(p, "%02x", buf[i]);
|
sprintf(p, "%02x", buf[i]);
|
||||||
*p = '\0';
|
*p = '\0';
|
||||||
data = data_append(data, name, "", DATA_STRING, tstr, NULL);
|
data = data_append(data, name, "", DATA_STRING, tstr, NULL);
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue