chiark / gitweb /
hwdb: fix dangling 'else' ambuguity
authorDavid Herrmann <dh.herrmann@gmail.com>
Fri, 12 Dec 2014 08:43:54 +0000 (09:43 +0100)
committerDavid Herrmann <dh.herrmann@gmail.com>
Fri, 12 Dec 2014 08:43:54 +0000 (09:43 +0100)
commitbff686e2a981ccd0888cdf1981977d24320f1770
tree985439518c0dd5c3b3b7f9361d604e743c1d41ec
parent0b9315719154fa7ea1a075e1b3785713f5acc3a1
hwdb: fix dangling 'else' ambuguity

Imagine the following use of hwdb:

    if (condition_A)
        SD_HWDB_FOREACH_PROPERTY(hwdb, modalias, key, value)
            operation_A(key, value);
    else
        log_error("...");

This should work just fine, but but definitely does not what you would
expect. Due to how SD_HWDB_FOREACH_PROPERTY is defined, the dangling
'else' is linked to the hidden 'if' statement in the macro instead of the
outer 'if (condition_A)'. This is unexpected and really annoying to debug.

Fix this by never leaving un-finished if-statements in
SD_HWDB_FOREACH_PROPERTY(). We simply inverse the if() statement and
explicitly add an 'else'-branch. This way, the statement is closed and all
ambuguities are resolved.
src/systemd/sd-hwdb.h