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>
Wed, 6 May 2009 16:51:16 +0000 (12:51 -0400)
commitb5a1f3c54a48ea3079622b0ec3023c79b95ed135
tree42b90261f85a144af84395e571239f8734c3fb36
parentb2f943e9c0d5ae1d6293d418b0c0b2a03799bb84
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