diff options
| author | David Herrmann <dh.herrmann@gmail.com> | 2015-07-28 16:46:51 +0200 |
|---|---|---|
| committer | David Herrmann <dh.herrmann@gmail.com> | 2015-07-29 13:03:09 +0200 |
| commit | a388569f609017430133c1c6584eca34cbff3aa3 (patch) | |
| tree | cdc628a420d2914e4e13e31af38f95ca2fe1c97f /src/libsystemd/sd-bus/bus-message.c | |
| parent | 39d1a2d05d4cc75c57de2658ceef04caa9928768 (diff) | |
sd-bus: don't assert() on valid signatures
Right now sd_bus_message_skip() will abort execution if passed a
signature of the unary type "()". Regardless whether this should be
supported or not, we really must not abort. Drop the incorrect assertion
and add a test-case for this.
Diffstat (limited to 'src/libsystemd/sd-bus/bus-message.c')
| -rw-r--r-- | src/libsystemd/sd-bus/bus-message.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/libsystemd/sd-bus/bus-message.c b/src/libsystemd/sd-bus/bus-message.c index 18685be8ff..081f39d83b 100644 --- a/src/libsystemd/sd-bus/bus-message.c +++ b/src/libsystemd/sd-bus/bus-message.c @@ -4756,7 +4756,6 @@ _public_ int sd_bus_message_skip(sd_bus_message *m, const char *types) { r = sd_bus_message_skip(m, s); if (r < 0) return r; - assert(r != 0); r = sd_bus_message_exit_container(m); if (r < 0) |