diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2018-10-29 10:27:04 +0100 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2018-10-29 10:27:04 +0100 |
| commit | 63af44e30ffdae8be2a9ece095ef5d7c5ed90713 (patch) | |
| tree | 1c0b3e315197df19edfbd12aadf8739a678254de | |
| parent | 0c9b09dff4913134e3e99bbdbef00a71c5e5d7e3 (diff) | |
| download | curl-bagder/unit1650-null-pointer-memcmp.tar.gz | |
unit1650: fix "null pointer passed as argument 1 to memcmp"bagder/unit1650-null-pointer-memcmp
Detected by UndefinedBehaviorSanitizer
| -rw-r--r-- | tests/unit/unit1650.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/unit/unit1650.c b/tests/unit/unit1650.c index 723c06421..ddf228f8d 100644 --- a/tests/unit/unit1650.c +++ b/tests/unit/unit1650.c @@ -170,7 +170,7 @@ UNITTEST_START fprintf(stderr, "DNS encode made: %s\n", hexdump(buffer, size)); return 2; } - else if(memcmp(req[i].packet, buffer, size)) { + else if(req[i].packet && memcmp(req[i].packet, buffer, size)) { fprintf(stderr, "DNS encode made: %s\n", hexdump(buffer, size)); fprintf(stderr, "... instead of: %s\n", hexdump((unsigned char *)req[i].packet, size)); |