Bug 17803 - Fix both test case and validation logic
authorColin Walters <walters@verbum.org>
Wed, 1 Apr 2009 16:02:00 +0000 (12:02 -0400)
committerColin Walters <walters@verbum.org>
Fri, 17 Apr 2009 17:25:39 +0000 (13:25 -0400)
commite8f8c1c5a2bddfbf43c168323c9c9fd78f51a643
treee726f50fa27e03dde3479619cceea75db1b57906
parent8f5c3e3c25901b089bc3fb87d53c7a98df48121f
Bug 17803 - Fix both test case and validation logic

The previous commit had errors in both the test case and
the validation logic.  The test case was missing a trailing
comma before the previous one, so we weren't testing the
signature we thought we were.

The validation logic was wrong because if the type was not valid,
we'd drop through the entire if clause, and thus skip returning
an error code, and accept the signature.
dbus/dbus-marshal-validate-util.c
dbus/dbus-marshal-validate.c