tree b7c5587c205c70070d8999ce9f10e0b49f4969ea
parent 376575018d05edc3d060f84b5909d0b6866e74bf
author William A. Kennington III <wak@google.com> 1529721348 -0700
committer Brad Bishop <bradleyb@fuzziesquirrel.com> 1531160442 +0000

SdBusError: Really own the sd_bus_error

The sd_bus_error constructor claims to own the sd_bus_error pointer but
it doesn't prevent the user from trivially re-using it in a broken way.
Instead of accepting the argument by value, take the old value and
overwrite the caller's copy to NULL it out. This way future calls to
sd_bus_error_free() by the caller will do the right thing.

Tested:
    Builds still work and tests are passing.

Change-Id: I0afd856f0a2a08a08f25fd43c051aae4b2a645f4
Signed-off-by: William A. Kennington III <wak@google.com>