mirror of
https://gitlab.xiph.org/xiph/ezstream.git
synced 2024-12-04 14:46:31 -05:00
Assert errstr2 to be not NULL before dereferencing it
This commit is contained in:
parent
1790ca5508
commit
3a471dc164
@ -252,6 +252,7 @@ START_TEST(test_server_protocol)
|
|||||||
|
|
||||||
errstr2 = NULL;
|
errstr2 = NULL;
|
||||||
ck_assert_int_eq(cfg_set_server_protocol("invalid", &errstr2), -1);
|
ck_assert_int_eq(cfg_set_server_protocol("invalid", &errstr2), -1);
|
||||||
|
ck_assert_ptr_ne(errstr2, NULL);
|
||||||
ck_assert_str_eq(errstr2, "unsupported");
|
ck_assert_str_eq(errstr2, "unsupported");
|
||||||
|
|
||||||
ck_assert_int_eq(cfg_set_server_protocol("hTtP", NULL), 0);
|
ck_assert_int_eq(cfg_set_server_protocol("hTtP", NULL), 0);
|
||||||
@ -500,27 +501,32 @@ START_TEST(test_metadata_format_str)
|
|||||||
errstr2 = NULL;
|
errstr2 = NULL;
|
||||||
ck_assert_int_eq(cfg_set_metadata_format_str(PLACEHOLDER_METADATA,
|
ck_assert_int_eq(cfg_set_metadata_format_str(PLACEHOLDER_METADATA,
|
||||||
&errstr2), -1);
|
&errstr2), -1);
|
||||||
|
ck_assert_ptr_ne(errstr2, NULL);
|
||||||
ck_assert_str_eq(errstr2,
|
ck_assert_str_eq(errstr2,
|
||||||
"prohibited placeholder " PLACEHOLDER_METADATA);
|
"prohibited placeholder " PLACEHOLDER_METADATA);
|
||||||
|
|
||||||
errstr2 = NULL;
|
errstr2 = NULL;
|
||||||
ck_assert_int_eq(cfg_set_metadata_format_str(
|
ck_assert_int_eq(cfg_set_metadata_format_str(
|
||||||
PLACEHOLDER_TRACK PLACEHOLDER_TRACK, &errstr2), -1);
|
PLACEHOLDER_TRACK PLACEHOLDER_TRACK, &errstr2), -1);
|
||||||
|
ck_assert_ptr_ne(errstr2, NULL);
|
||||||
ck_assert_str_eq(errstr2, "duplicate placeholder " PLACEHOLDER_TRACK);
|
ck_assert_str_eq(errstr2, "duplicate placeholder " PLACEHOLDER_TRACK);
|
||||||
|
|
||||||
errstr2 = NULL;
|
errstr2 = NULL;
|
||||||
ck_assert_int_eq(cfg_set_metadata_format_str(
|
ck_assert_int_eq(cfg_set_metadata_format_str(
|
||||||
PLACEHOLDER_STRING PLACEHOLDER_STRING, &errstr2), -1);
|
PLACEHOLDER_STRING PLACEHOLDER_STRING, &errstr2), -1);
|
||||||
|
ck_assert_ptr_ne(errstr2, NULL);
|
||||||
ck_assert_str_eq(errstr2, "duplicate placeholder " PLACEHOLDER_STRING);
|
ck_assert_str_eq(errstr2, "duplicate placeholder " PLACEHOLDER_STRING);
|
||||||
|
|
||||||
errstr2 = NULL;
|
errstr2 = NULL;
|
||||||
ck_assert_int_eq(cfg_set_metadata_format_str(
|
ck_assert_int_eq(cfg_set_metadata_format_str(
|
||||||
PLACEHOLDER_ARTIST PLACEHOLDER_ARTIST, &errstr2), -1);
|
PLACEHOLDER_ARTIST PLACEHOLDER_ARTIST, &errstr2), -1);
|
||||||
|
ck_assert_ptr_ne(errstr2, NULL);
|
||||||
ck_assert_str_eq(errstr2, "duplicate placeholder " PLACEHOLDER_ARTIST);
|
ck_assert_str_eq(errstr2, "duplicate placeholder " PLACEHOLDER_ARTIST);
|
||||||
|
|
||||||
errstr2 = NULL;
|
errstr2 = NULL;
|
||||||
ck_assert_int_eq(cfg_set_metadata_format_str(
|
ck_assert_int_eq(cfg_set_metadata_format_str(
|
||||||
PLACEHOLDER_TITLE PLACEHOLDER_TITLE, &errstr2), -1);
|
PLACEHOLDER_TITLE PLACEHOLDER_TITLE, &errstr2), -1);
|
||||||
|
ck_assert_ptr_ne(errstr2, NULL);
|
||||||
ck_assert_str_eq(errstr2, "duplicate placeholder " PLACEHOLDER_TITLE);
|
ck_assert_str_eq(errstr2, "duplicate placeholder " PLACEHOLDER_TITLE);
|
||||||
}
|
}
|
||||||
END_TEST
|
END_TEST
|
||||||
|
Loading…
Reference in New Issue
Block a user