]> xenbits.xensource.com Git - libvirt.git/commitdiff
tests: test negative number through dbus
authorEric Blake <eblake@redhat.com>
Fri, 9 Aug 2013 14:55:06 +0000 (08:55 -0600)
committerEric Blake <eblake@redhat.com>
Fri, 9 Aug 2013 14:57:01 +0000 (08:57 -0600)
Commit f1088c8 weakened a test, by not passing a value larger
than INT_MAX through an int slot.  Make the fix in a different
way, using an explicit negative value.  Suggested by Dan Berrange.

* tests/virdbustest.c (testMessageArray): Adjust previous fix.
(testMessageStruct): Use a negative number.

Signed-off-by: Eric Blake <eblake@redhat.com>
tests/virdbustest.c

index 528342bdd05c5f81d15aab7a7c0423882e4241cc..083202c1b5816f241956425f6e5df7fddeea7abe 100644 (file)
@@ -178,9 +178,9 @@ static int testMessageArray(const void *args ATTRIBUTE_UNUSED)
     DBusMessage *msg = NULL;
     int ret = -1;
     const char *in_str1 = "Hello";
-    int in_int32a = 100000000, out_int32a = 0;
-    int in_int32b = 200000000, out_int32b = 0;
-    int in_int32c = 300000000, out_int32c = 0;
+    int in_int32a = 1000000000, out_int32a = 0;
+    int in_int32b = 2000000000, out_int32b = 0;
+    int in_int32c = -2000000000, out_int32c = 0;
     const char *in_str2 = "World";
     char *out_str1 = NULL, *out_str2 = NULL;
 
@@ -236,7 +236,7 @@ static int testMessageStruct(const void *args ATTRIBUTE_UNUSED)
     unsigned short in_uint16 = 32000, out_uint16 = 0;
     int in_int32 = 100000000, out_int32 = 0;
     unsigned int in_uint32 = 200000000, out_uint32 = 0;
-    long long in_int64 = 1000000000000LL, out_int64 = 0;
+    long long in_int64 = -1000000000000LL, out_int64 = 0;
     unsigned long long in_uint64 = 2000000000000LL, out_uint64 = 0;
     double in_double = 3.14159265359, out_double = 0;;
     const char *in_string = "Hello World";