shared/bus-util: Don't replace exsting strv
Change the behavior of string arrays in a bus property map. Previously, passing the same strv pointer to more than one map entry would result in the old strv being freed and overwritten. With this change, an existing strv pointer is appended to. This is important if we want to create one strv comprised of multiple dependencies. This makes it so callers don't have to create one strv per dependency and subsequently merge them into one strv.
This commit is contained in:
parent
8ee08dc564
commit
411975ce63
|
@ -1127,7 +1127,7 @@ static int map_basic(sd_bus *bus, const char *member, sd_bus_message *m, unsigne
|
|||
if (r < 0)
|
||||
return r;
|
||||
|
||||
return strv_free_and_replace(*p, l);
|
||||
return strv_extend_strv(p, l, false);
|
||||
}
|
||||
|
||||
case SD_BUS_TYPE_BOOLEAN: {
|
||||
|
|
Loading…
Reference in a new issue