diff options
author | lurchi <lurchi@strangeplace.net> | 2019-06-27 12:59:05 +0200 |
---|---|---|
committer | lurchi <lurchi@strangeplace.net> | 2019-06-27 12:59:05 +0200 |
commit | 3ea320df4716a4ce16d738fa47bbe59740e36f42 (patch) | |
tree | c73de8a7095584cf24103bd3a87a6a49942dce73 /src/testbed | |
parent | 40a111b93ce2d38b875c2eda3e23d6b89a256599 (diff) | |
download | gnunet-3ea320df4716a4ce16d738fa47bbe59740e36f42.tar.gz gnunet-3ea320df4716a4ce16d738fa47bbe59740e36f42.zip |
new GNUNET_strlcpy allows using it more
Diffstat (limited to 'src/testbed')
-rw-r--r-- | src/testbed/gnunet-helper-testbed.c | 7 | ||||
-rw-r--r-- | src/testbed/gnunet-service-testbed.c | 8 | ||||
-rw-r--r-- | src/testbed/testbed_api_hosts.c | 16 |
3 files changed, 11 insertions, 20 deletions
diff --git a/src/testbed/gnunet-helper-testbed.c b/src/testbed/gnunet-helper-testbed.c index c56a795a6..687a54bcd 100644 --- a/src/testbed/gnunet-helper-testbed.c +++ b/src/testbed/gnunet-helper-testbed.c | |||
@@ -374,10 +374,9 @@ tokenizer_cb (void *cls, | |||
374 | if (0 != hostname_size) | 374 | if (0 != hostname_size) |
375 | { | 375 | { |
376 | hostname = GNUNET_malloc (hostname_size + 1); | 376 | hostname = GNUNET_malloc (hostname_size + 1); |
377 | /* intentionally use strncpy (hostname not null terminated) */ | 377 | GNUNET_strlcpy (hostname, |
378 | (void) strncpy (hostname, ((char *) &msg[1]) + trusted_ip_size + 1, | 378 | ((char *) &msg[1]) + trusted_ip_size + 1, |
379 | hostname_size); | 379 | hostname_size + 1); |
380 | hostname[hostname_size] = '\0'; | ||
381 | } | 380 | } |
382 | /* unset GNUNET_TESTING_PREFIX if present as it is more relevant for testbed */ | 381 | /* unset GNUNET_TESTING_PREFIX if present as it is more relevant for testbed */ |
383 | evstr = getenv (GNUNET_TESTING_PREFIX); | 382 | evstr = getenv (GNUNET_TESTING_PREFIX); |
diff --git a/src/testbed/gnunet-service-testbed.c b/src/testbed/gnunet-service-testbed.c index d740d31bc..289e17349 100644 --- a/src/testbed/gnunet-service-testbed.c +++ b/src/testbed/gnunet-service-testbed.c | |||
@@ -541,15 +541,11 @@ handle_add_host (void *cls, | |||
541 | if (0 != username_length) | 541 | if (0 != username_length) |
542 | { | 542 | { |
543 | username = GNUNET_malloc (username_length + 1); | 543 | username = GNUNET_malloc (username_length + 1); |
544 | /* intentionally use strncpy (message payload is not null terminated) */ | 544 | GNUNET_strlcpy (username, ptr, username_length + 1); |
545 | strncpy (username, ptr, username_length); | ||
546 | ptr += username_length; | 545 | ptr += username_length; |
547 | } | 546 | } |
548 | hostname = GNUNET_malloc (hostname_length + 1); | 547 | hostname = GNUNET_malloc (hostname_length + 1); |
549 | /* intentionally use strncpy (message payload is not null terminated) */ | 548 | GNUNET_strlcpy (hostname, ptr, hostname_length + 1); |
550 | strncpy (hostname, | ||
551 | ptr, | ||
552 | hostname_length); | ||
553 | if (NULL == (host_cfg = GNUNET_TESTBED_extract_config_ (&msg->header))) | 549 | if (NULL == (host_cfg = GNUNET_TESTBED_extract_config_ (&msg->header))) |
554 | { | 550 | { |
555 | GNUNET_free_non_null (username); | 551 | GNUNET_free_non_null (username); |
diff --git a/src/testbed/testbed_api_hosts.c b/src/testbed/testbed_api_hosts.c index d6521f766..6dbd99bc9 100644 --- a/src/testbed/testbed_api_hosts.c +++ b/src/testbed/testbed_api_hosts.c | |||
@@ -462,11 +462,9 @@ GNUNET_TESTBED_hosts_load_from_file (const char *filename, | |||
462 | { | 462 | { |
463 | size = pmatch[2].rm_eo - pmatch[2].rm_so; | 463 | size = pmatch[2].rm_eo - pmatch[2].rm_so; |
464 | username = GNUNET_malloc (size + 1); | 464 | username = GNUNET_malloc (size + 1); |
465 | /* | 465 | GNUNET_assert(0 != GNUNET_strlcpy (username, |
466 | * Intentionally use strncpy (buf is not necessarily null-terminated) | 466 | buf + pmatch[2].rm_so, |
467 | */ | 467 | size + 1)); |
468 | username[size] = '\0'; | ||
469 | GNUNET_assert (NULL != strncpy (username, buf + pmatch[2].rm_so, size)); | ||
470 | } | 468 | } |
471 | if (-1 != pmatch[5].rm_so) | 469 | if (-1 != pmatch[5].rm_so) |
472 | { | 470 | { |
@@ -474,11 +472,9 @@ GNUNET_TESTBED_hosts_load_from_file (const char *filename, | |||
474 | } | 472 | } |
475 | size = pmatch[3].rm_eo - pmatch[3].rm_so; | 473 | size = pmatch[3].rm_eo - pmatch[3].rm_so; |
476 | hostname = GNUNET_malloc (size + 1); | 474 | hostname = GNUNET_malloc (size + 1); |
477 | /* | 475 | GNUNET_assert(0 != GNUNET_strlcpy (hostname, |
478 | * Intentionally use strncpy (buf is not necessarily null-terminated) | 476 | buf + pmatch[3].rm_so, |
479 | */ | 477 | size + 1)); |
480 | hostname[size] = '\0'; | ||
481 | GNUNET_assert (NULL != strncpy (hostname, buf + pmatch[3].rm_so, size)); | ||
482 | LOG (GNUNET_ERROR_TYPE_DEBUG, | 478 | LOG (GNUNET_ERROR_TYPE_DEBUG, |
483 | "Successfully read host %s, port %d and user %s from file\n", | 479 | "Successfully read host %s, port %d and user %s from file\n", |
484 | (NULL == hostname) ? "NULL" : hostname, | 480 | (NULL == hostname) ? "NULL" : hostname, |