]> granicus.if.org Git - check/commitdiff
Pass in proper arguments for upack in test_pack_abuse
authorbrarcher <brarcher@64e312b2-a51f-0410-8e61-82d0ca0eb02a>
Mon, 23 Jun 2014 04:13:44 +0000 (04:13 +0000)
committerbrarcher <brarcher@64e312b2-a51f-0410-8e61-82d0ca0eb02a>
Mon, 23 Jun 2014 04:13:44 +0000 (04:13 +0000)
Even those the test 'abuses' upack, passing in a non-pointer
to a pointer argument is a bad idea.

git-svn-id: svn+ssh://svn.code.sf.net/p/check/code/trunk@1164 64e312b2-a51f-0410-8e61-82d0ca0eb02a

tests/check_check_pack.c

index afadaaa9b85d6760d30da432849587d66af43402..596d40e9086dc101aa0e7a2f975780630745840f 100644 (file)
@@ -138,6 +138,7 @@ START_TEST(test_pack_abuse)
 {
   char *buf;
   CtxMsg cmsg;
+  enum ck_msg_type type;
 
   /* Should report -1 (e.g. invalid) if no buffer is passed */
   ck_assert_int_eq(pack(CK_MSG_CTX, NULL, (CheckMsg *) &cmsg), -1);
@@ -146,7 +147,7 @@ START_TEST(test_pack_abuse)
   ck_assert_int_eq(pack(CK_MSG_CTX, &buf, NULL), 0);
 
   /* Should report -1 (e.g. invalid) if no buffer is passed */
-  ck_assert_int_eq(upack(NULL, (CheckMsg *) &cmsg, CK_MSG_CTX), -1);
+  ck_assert_int_eq(upack(NULL, (CheckMsg *) &cmsg, &type), -1);
 }
 END_TEST