diff options
| author | tony.nguyen@bt.com <tony.nguyen@bt.com> | 2019-06-08 05:25:52 +0000 |
|---|---|---|
| committer | Laurent Vivier <laurent@vivier.eu> | 2019-08-21 10:27:13 +0200 |
| commit | 348fbd5816cab30d38d7a3727a4b83404797128a (patch) | |
| tree | 13f4a9c42f28f95e7a87d2e0da64f246b278b5b5 | |
| parent | 17dc57990320edaad52ac9ea808be9719c91cea6 (diff) | |
test: Use g_strndup instead of plain strndup
Due to memory management rules. See HACKING.
Signed-off-by: Tony Nguyen <tony.nguyen@bt.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <dce313b46d294ada8826d34609a3447e@tpw09926dag18e.domain1.systemhost.net>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
| -rw-r--r-- | tests/check-qjson.c | 2 | ||||
| -rw-r--r-- | tests/migration/stress.c | 8 |
2 files changed, 5 insertions, 5 deletions
diff --git a/tests/check-qjson.c b/tests/check-qjson.c index fa2afccb0a..07a773e653 100644 --- a/tests/check-qjson.c +++ b/tests/check-qjson.c @@ -767,7 +767,7 @@ static void utf8_string(void) if (*end == ' ') { end++; } - in = strndup(tail, end - tail); + in = g_strndup(tail, end - tail); str = from_json_str(in, j, NULL); g_assert(!str); g_free(in); diff --git a/tests/migration/stress.c b/tests/migration/stress.c index 49a03aab7b..d9aa4afe92 100644 --- a/tests/migration/stress.c +++ b/tests/migration/stress.c @@ -104,9 +104,9 @@ static int get_command_arg_str(const char *name, } if (end) - *val = strndup(start, end - start); + *val = g_strndup(start, end - start); else - *val = strdup(start); + *val = g_strdup(start); return 1; } @@ -126,10 +126,10 @@ static int get_command_arg_ull(const char *name, if (errno || *end) { fprintf(stderr, "%s (%05d): ERROR: cannot parse %s value %s\n", argv0, gettid(), name, valstr); - free(valstr); + g_free(valstr); return -1; } - free(valstr); + g_free(valstr); return 0; } |