mirrors/pipewire
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/pipewire.git synced 2026-06-16 14:33:04 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

treewide: avoid unitialized spa_dict::flags

Browse source 
In multiple cases the `flags` member of `spa_dict` is left unitialized,
so try to avoid that. For example in `fill_node_info_proplist()` it is
accessed in `spa_dict_lookup_item()`.

This also modifies `collect_props()` to not depend on a partially initialized
`dict` parameter.
This commit is contained in:
Barnabás Pőcze 2026-05-10 16:44:17 +02:00 • committed by Wim Taymans
parent 4c86dd53fc
commit 5e17161d0f
4 changed files with 13 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

3
spa/plugins/alsa/acp/proplist.h
View file

@ -180,8 +180,7 @@ static inline pa_proplist* pa_proplist_new_dict(const struct acp_dict *dict)
static inline void pa_proplist_as_dict(const pa_proplist *p, struct acp_dict *dict) static inline void pa_proplist_as_dict(const pa_proplist *p, struct acp_dict *dict)
{ {
dict->n_items = pa_proplist_size(p); *dict = ACP_DICT_INIT(p->array.data, pa_proplist_size(p));
dict->items = p->array.data;
} }
#ifdef __cplusplus #ifdef __cplusplus

12
src/modules/module-protocol-pulse/collect.c
View file

@ -502,7 +502,8 @@ uint32_t find_port_index(struct pw_manager_object *card, uint32_t direction, con
return SPA_ID_INVALID; return SPA_ID_INVALID;
} }
struct spa_dict *collect_props(struct spa_pod *info, struct spa_dict *dict) struct spa_dict *collect_props(struct spa_pod *info, struct spa_dict *dict,
struct spa_dict_item *items, size_t capacity)
{ {
struct spa_pod_parser prs; struct spa_pod_parser prs;
struct spa_pod_frame f[1]; struct spa_pod_frame f[1];
@ -513,15 +514,18 @@ struct spa_dict *collect_props(struct spa_pod *info, struct spa_dict *dict)
spa_pod_parser_get_int(&prs, &n_items) < 0) spa_pod_parser_get_int(&prs, &n_items) < 0)
return NULL; return NULL;
if (n_items < 0 || (size_t) n_items > capacity)
return NULL;
for (n = 0; n < n_items; n++) { for (n = 0; n < n_items; n++) {
if (spa_pod_parser_get(&prs, if (spa_pod_parser_get(&prs,
SPA_POD_String(&dict->items[n].key), SPA_POD_String(&items[n].key),
SPA_POD_String(&dict->items[n].value), SPA_POD_String(&items[n].value),
NULL) < 0) NULL) < 0)
break; break;
} }
spa_pod_parser_pop(&prs, &f[0]); spa_pod_parser_pop(&prs, &f[0]);
dict->n_items = n; *dict = SPA_DICT_INIT(items, n);
return dict; return dict;
} }

3
src/modules/module-protocol-pulse/collect.h
View file

@ -142,7 +142,8 @@ uint32_t collect_transport_codec_info(struct pw_manager_object *card,
/* ========================================================================== */ /* ========================================================================== */
struct spa_dict *collect_props(struct spa_pod *info, struct spa_dict *dict); struct spa_dict *collect_props(struct spa_pod *info, struct spa_dict *dict,
struct spa_dict_item *items, size_t capacity);
uint32_t find_profile_index(struct pw_manager_object *card, const char *name); uint32_t find_profile_index(struct pw_manager_object *card, const char *name);
uint32_t find_port_index(struct pw_manager_object *card, uint32_t direction, const char *port_name); uint32_t find_port_index(struct pw_manager_object *card, uint32_t direction, const char *port_name);
struct pw_manager_object *find_peer_for_link(struct pw_manager *m, struct pw_manager_object *find_peer_for_link(struct pw_manager *m,

5
src/modules/module-protocol-pulse/pulse-server.c
View file

@ -3282,7 +3282,7 @@ static int do_update_proplist(struct client *client, uint32_t command, uint32_t
static int do_remove_proplist(struct client *client, uint32_t command, uint32_t tag, struct message *m) static int do_remove_proplist(struct client *client, uint32_t command, uint32_t tag, struct message *m)
{ {
uint32_t i, channel; uint32_t i, channel;
struct spa_dict dict; struct spa_dict dict = SPA_DICT_INIT(NULL, 0);
struct spa_dict_item *items; struct spa_dict_item *items;
spa_autoptr(pw_properties) props = pw_properties_new(NULL, NULL); spa_autoptr(pw_properties) props = pw_properties_new(NULL, NULL);
@ -3643,8 +3643,7 @@ static int fill_card_info(struct client *client, struct message *m,
if (pi->info && pi->n_props > 0) { if (pi->info && pi->n_props > 0) {
items = alloca(pi->n_props * sizeof(*items)); items = alloca(pi->n_props * sizeof(*items));
dict.items = items; pdict = collect_props(pi->info, &dict, items, pi->n_props);
pdict = collect_props(pi->info, &dict);
} }
message_put(m, message_put(m,