Commit 348fbd58 authored by tony.nguyen@bt.com's avatar tony.nguyen@bt.com Committed by Laurent Vivier
Browse files

test: Use g_strndup instead of plain strndup 



Due to memory management rules. See HACKING.

Signed-off-by: default avatarTony Nguyen <tony.nguyen@bt.com>
Reviewed-by: default avatarMarkus Armbruster <armbru@redhat.com>

Message-Id: <dce313b46d294ada8826d34609a3447e@tpw09926dag18e.domain1.systemhost.net>
Signed-off-by: default avatarLaurent Vivier <laurent@vivier.eu>
parent 17dc5799

tests/check-qjson.c

+1 −1
Original line number Diff line number Diff line
@@ -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);

tests/migration/stress.c

+4 −4
Original line number Diff line number Diff line
@@ -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;

}