diff options
| author | Michael Tuexen <tuexen@fh-muenster.de> | 2020-05-18 14:09:04 +0200 |
|---|---|---|
| committer | Michael Tuexen <tuexen@fh-muenster.de> | 2020-05-18 14:09:04 +0200 |
| commit | d98d2c4c6f4a6a8cafef2a6a2c81de0137a07da0 (patch) | |
| tree | 600285fcf4330386fd6cc98a0dc63f44e09be67b /programs/http_client_upcall.c | |
| parent | a05154264872ec8e5d4143ac841a306f54ace231 (diff) | |
| download | usrsctp-d98d2c4c6f4a6a8cafef2a6a2c81de0137a07da0.tar.gz | |
Check return value of snprintf().
This should fix https://github.com/sctplab/usrsctp/issues/453.
Diffstat (limited to 'programs/http_client_upcall.c')
| -rw-r--r-- | programs/http_client_upcall.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/programs/http_client_upcall.c b/programs/http_client_upcall.c index 5583b54c..63916266 100644 --- a/programs/http_client_upcall.c +++ b/programs/http_client_upcall.c @@ -276,16 +276,20 @@ main(int argc, char *argv[]) if (argc > 6) { #ifdef _WIN32 - _snprintf(request, sizeof(request), "%s %s %s", request_prefix, argv[6], request_postfix); + if (_snprintf(request, sizeof(request), "%s %s %s", request_prefix, argv[6], request_postfix) < 0) { #else - snprintf(request, sizeof(request), "%s %s %s", request_prefix, argv[6], request_postfix); + if (snprintf(request, sizeof(request), "%s %s %s", request_prefix, argv[6], request_postfix) < 0) { #endif + request[0] = '\0'; + } } else { #ifdef _WIN32 - _snprintf(request, sizeof(request), "%s %s %s", request_prefix, "/", request_postfix); + if (_snprintf(request, sizeof(request), "%s %s %s", request_prefix, "/", request_postfix) < 0) { #else - snprintf(request, sizeof(request), "%s %s %s", request_prefix, "/", request_postfix); + if (snprintf(request, sizeof(request), "%s %s %s", request_prefix, "/", request_postfix) < 0) { #endif + request[0] = '\0'; + } } printf("\nHTTP request:\n%s\n", request); |