]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s4:torture: Initialize struct wrepl_wins_name
authorAndreas Schneider <asn@samba.org>
Mon, 24 Jun 2024 13:19:20 +0000 (15:19 +0200)
committerAndreas Schneider <asn@cryptomilk.org>
Fri, 28 Jun 2024 12:44:31 +0000 (12:44 +0000)
"Error: UNINIT (CWE-457):
samba-4.20.0rc2/source4/torture/nbt/winsreplication.c:1244: var_decl: Declaring variable ""wins_name1"" without initializer.
samba-4.20.0rc2/source4/torture/nbt/winsreplication.c:4835: assign: Assigning: ""wins_name_r1"" = ""&wins_name1"", which points to uninitialized data.
samba-4.20.0rc2/source4/torture/nbt/winsreplication.c:4905: uninit_use_in_call: Using uninitialized value ""*wins_name_r1"". Field ""wins_name_r1->is_group"" is uninitialized when calling ""test_wrepl_update_one"".
 4903|
 4904|    /* now apply R1 */
 4905|->  ret &= test_wrepl_update_one(tctx, ctx, records[i].r1.owner, wins_name_r1);
 4906|    ret &= test_wrepl_is_applied(tctx, ctx, records[i].r1.owner,
 4907|         wins_name_r1, records[i].r1.apply_expected);"

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Noel Power <npower@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Fri Jun 28 12:44:31 UTC 2024 on atb-devel-224

source4/torture/nbt/winsreplication.c

index 2bfdb84daed308d7926a9e607bfa48043a562f40..77bef8c81e1ba61676e15177f223d92304fb127b 100644 (file)
@@ -1241,8 +1241,12 @@ static bool test_conflict_different_owner(struct torture_context *tctx,
                                                                                  struct test_wrepl_conflict_conn *ctx)
 {
        bool ret = true;
-       struct wrepl_wins_name wins_name1;
-       struct wrepl_wins_name wins_name2;
+       struct wrepl_wins_name wins_name1 = {
+               .flags = 0,
+       };
+       struct wrepl_wins_name wins_name2 = {
+               .flags = 0,
+       };
        struct wrepl_wins_name *wins_name_r1;
        struct wrepl_wins_name *wins_name_r2;
        uint32_t i;