[PATCH] introspect: Fix ABI break introduce by b98a2e1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>

The size of pa_card_profile_info cannot change even if it just a field
appended to end because it is appended to an array this may lead clients
to access invalid data.

To fix a new struct called pa_card_profile_info2 is introduced and shall
be used for now on while pa_card_profile_info shall be considered
deprecated but it is still mantained for backward compatibility.
---
 src/pulse/introspect.c | 88 +++++++++++++++++++++++++++++++++++---------------
 src/pulse/introspect.h | 14 +++++++-
 src/utils/pactl.c      |  4 +--
 3 files changed, 77 insertions(+), 29 deletions(-)

diff --git a/src/pulse/introspect.c b/src/pulse/introspect.c
index 45e0115..266f03f 100644
--- a/src/pulse/introspect.c
+++ b/src/pulse/introspect.c
@@ -768,6 +768,7 @@ static void card_info_free(pa_card_info* i) {
         pa_proplist_free(i->proplist);
 
     pa_xfree(i->profiles);
+    pa_xfree(i->profiles2);
 
     if (i->ports) {
         uint32_t j;
@@ -776,6 +777,8 @@ static void card_info_free(pa_card_info* i) {
             if (i->ports[j]) {
                 if (i->ports[j]->profiles)
                     pa_xfree(i->ports[j]->profiles);
+                if (i->ports[j]->profiles2)
+                    pa_xfree(i->ports[j]->profiles2);
                 if (i->ports[j]->proplist)
                     pa_proplist_free(i->ports[j]->proplist);
             }
@@ -837,7 +840,12 @@ static int fill_card_port_info(pa_context *context, pa_tagstruct* t, pa_card_inf
                     return -PA_ERR_PROTOCOL;
 
                 for (l = 0; l < i->n_profiles; l++) {
-                    if (pa_streq(i->profiles[l].name, profilename)) {
+                    if (context->version >= 29) {
+                        if (pa_streq(i->profiles2[l].name, profilename)) {
+                            port->profiles2[k] = &i->profiles2[l];
+                            break;
+                        }
+                    } else if (pa_streq(i->profiles[l].name, profilename)) {
                         port->profiles[k] = &i->profiles[l];
                         break;
                     }
@@ -857,6 +865,49 @@ static int fill_card_port_info(pa_context *context, pa_tagstruct* t, pa_card_inf
     return 0;
 }
 
+static int fill_card_profile_info(pa_context *context, pa_tagstruct* t, pa_card_info* i) {
+    uint32_t j;
+
+    i->profiles = pa_xnew0(pa_card_profile_info, i->n_profiles+1);
+
+    if (context->version >= 29)
+        i->profiles2 = pa_xnew0(pa_card_profile_info2, i->n_profiles+1);
+
+    for (j = 0; j < i->n_profiles; j++) {
+        if (pa_tagstruct_gets(t, &i->profiles[j].name) < 0 ||
+            pa_tagstruct_gets(t, &i->profiles[j].description) < 0 ||
+            pa_tagstruct_getu32(t, &i->profiles[j].n_sinks) < 0 ||
+            pa_tagstruct_getu32(t, &i->profiles[j].n_sources) < 0 ||
+            pa_tagstruct_getu32(t, &i->profiles[j].priority) < 0)
+                return -PA_ERR_PROTOCOL;
+
+        if (context->version >= 29) {
+            uint32_t av;
+
+            i->profiles2[j].name = i->profiles[j].name;
+            i->profiles2[j].description = i->profiles[j].description;
+            i->profiles2[j].n_sinks = i->profiles[j].n_sinks;
+            i->profiles2[j].n_sources = i->profiles[j].n_sources;
+            i->profiles2[j].priority = i->profiles[j].priority;
+
+            if (pa_tagstruct_getu32(t, &av) < 0)
+                return -PA_ERR_PROTOCOL;
+
+            i->profiles2[j].available = av;
+        }
+    }
+
+    /* Terminate with an extra NULL entry, just to make sure */
+    i->profiles[j].name = NULL;
+    i->profiles[j].description = NULL;
+
+    if (context->version >= 29)
+        i->profiles2[j].name = NULL;
+        i->profiles2[j].description = NULL;
+
+    return 0;
+}
+
 static void context_get_card_info_callback(pa_pdispatch *pd, uint32_t command, uint32_t tag, pa_tagstruct *t, void *userdata) {
     pa_operation *o = userdata;
     int eol = 1;
@@ -890,29 +941,8 @@ static void context_get_card_info_callback(pa_pdispatch *pd, uint32_t command, u
                     goto fail;
 
             if (i.n_profiles > 0) {
-                i.profiles = pa_xnew0(pa_card_profile_info, i.n_profiles+1);
-
-                for (j = 0; j < i.n_profiles; j++) {
-
-                    if (pa_tagstruct_gets(t, &i.profiles[j].name) < 0 ||
-                        pa_tagstruct_gets(t, &i.profiles[j].description) < 0 ||
-                        pa_tagstruct_getu32(t, &i.profiles[j].n_sinks) < 0 ||
-                        pa_tagstruct_getu32(t, &i.profiles[j].n_sources) < 0 ||
-                        pa_tagstruct_getu32(t, &i.profiles[j].priority) < 0)
-                            goto fail;
-
-                    i.profiles[j].available = 1;
-                    if (o->context->version >= 29) {
-                        uint32_t av;
-                        if (pa_tagstruct_getu32(t, &av) < 0)
-                            goto fail;
-                        i.profiles[j].available = av;
-                    }
-                }
-
-                /* Terminate with an extra NULL entry, just to make sure */
-                i.profiles[j].name = NULL;
-                i.profiles[j].description = NULL;
+                if (fill_card_profile_info(o->context, t, &i) < 0)
+                    goto fail;
             }
 
             i.proplist = pa_proplist_new();
@@ -926,11 +956,17 @@ static void context_get_card_info_callback(pa_pdispatch *pd, uint32_t command, u
             }
 
             if (ap) {
-                for (j = 0; j < i.n_profiles; j++)
-                    if (pa_streq(i.profiles[j].name, ap)) {
+                for (j = 0; j < i.n_profiles; j++) {
+                    if (o->context->version >= 29) {
+                        if (pa_streq(i.profiles2[j].name, ap)) {
+                            i.active_profile2 = &i.profiles2[j];
+                            break;
+                        }
+                    } else if (pa_streq(i.profiles[j].name, ap)) {
                         i.active_profile = &i.profiles[j];
                         break;
                     }
+                }
             }
 
             if (o->context->version >= 26) {
diff --git a/src/pulse/introspect.h b/src/pulse/introspect.h
index f199a18..8040eb3 100644
--- a/src/pulse/introspect.h
+++ b/src/pulse/introspect.h
@@ -452,13 +452,22 @@ typedef struct pa_card_profile_info {
     uint32_t n_sinks;                   /**< Number of sinks this profile would create */
     uint32_t n_sources;                 /**< Number of sources this profile would create */
     uint32_t priority;                  /**< The higher this value is, the more useful this profile is as a default. */
+} pa_card_profile_info;
+
+/** Stores extended information about a specific profile of a card. \since 5.0 */
+typedef struct pa_card_profile_info2 {
+    const char *name;                   /**< Name of this profile */
+    const char *description;            /**< Description of this profile */
+    uint32_t n_sinks;                   /**< Number of sinks this profile would create */
+    uint32_t n_sources;                 /**< Number of sources this profile would create */
+    uint32_t priority;                  /**< The higher this value is, the more useful this profile is as a default. */
     int available;
     /**< Is this profile available? If this is zero, meaning "unavailable",
      * then it makes no sense to try to activate this profile. If this is
      * non-zero, it's still not a guarantee that activating the profile will
      * result in anything useful, it just means that the server isn't aware of
      * any reason why the profile would definitely be useless. \since 5.0 */
-} pa_card_profile_info;
+} pa_card_profile_info2;
 
 /** Stores information about a specific port of a card.  Please
  * note that this structure can be extended as part of evolutionary
@@ -471,6 +480,7 @@ typedef struct pa_card_port_info {
     int direction;                      /**< A #pa_direction enum, indicating the direction of this port. */
     uint32_t n_profiles;                /**< Number of entries in profile array */
     pa_card_profile_info** profiles;    /**< Array of pointers to available profiles, or NULL. Array is terminated by an entry set to NULL. */
+    pa_card_profile_info2** profiles2;  /**< Array of pointers to available profiles, or NULL. Array is terminated by an entry set to NULL. */
     pa_proplist *proplist;              /**< Property list */
     int64_t latency_offset;             /**< Latency offset of the port that gets added to the sink/source latency when the port is active. \since 3.0 */
 } pa_card_port_info;
@@ -485,7 +495,9 @@ typedef struct pa_card_info {
     const char *driver;                  /**< Driver name */
     uint32_t n_profiles;                 /**< Number of entries in profile array */
     pa_card_profile_info* profiles;      /**< Array of available profile, or NULL. Array is terminated by an entry with name set to NULL. Number of entries is stored in n_profiles. */
+    pa_card_profile_info2* profiles2;    /**< Array of available profile, or NULL. Array is terminated by an entry with name set to NULL. Number of entries is stored in n_profiles. */
     pa_card_profile_info* active_profile; /**< Pointer to active profile in the array, or NULL. */
+    pa_card_profile_info2* active_profile2; /**< Pointer to active profile in the array, or NULL. */
     pa_proplist *proplist;               /**< Property list */
     uint32_t n_ports;                    /**< Number of entries in port array */
     pa_card_port_info **ports;           /**< Array of pointers to ports, or NULL. Array is terminated by an entry set to NULL. */
diff --git a/src/utils/pactl.c b/src/utils/pactl.c
index df47caa..ed90b84 100644
--- a/src/utils/pactl.c
+++ b/src/utils/pactl.c
@@ -571,10 +571,10 @@ static void get_card_info_callback(pa_context *c, const pa_card_info *i, int is_
     pa_xfree(pl);
 
     if (i->profiles) {
-        pa_card_profile_info *p;
+        pa_card_profile_info2 *p;
 
         printf(_("\tProfiles:\n"));
-        for (p = i->profiles; p->name; p++)
+        for (p = i->profiles2; p->name; p++)
             printf("\t\t%s: %s (sinks: %u, sources: %u, priority: %u, available: %s)\n", p->name,
                 p->description, p->n_sinks, p->n_sources, p->priority, pa_yes_no(p->available));
     }
-- 
1.8.3.1



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux