lkml.org 
[lkml]   [2022]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.18 129/879] ASoC: SOF: ipc3-topology: Correct get_control_data for non bytes payload
    Date
    From: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>

    [ Upstream commit a962890a5a3cce903ff7c7a19fadee63ed9efdc7 ]

    It is possible to craft a topology where sof_get_control_data() would do
    out of bounds access because it expects that it is only called when the
    payload is bytes type.
    Confusingly it also handles other types of controls, but the payload
    parsing implementation is only valid for bytes.

    Fix the code to count the non bytes controls and instead of storing a
    pointer to sof_abi_hdr in sof_widget_data (which is only valid for bytes),
    store the pointer to the data itself and add a new member to save the size
    of the data.

    In case of non bytes controls we store the pointer to the chanv itself,
    which is just an array of values at the end.

    In case of bytes control, drop the wrong cdata->data (wdata[i].pdata) check
    against NULL since it is incorrect and invalid in this context.
    The data is pointing to the end of cdata struct, so it should never be
    null.

    Reported-by: Sergey Senozhatsky <senozhatsky@chromium.org>
    Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
    Reviewed-by: Sergey Senozhatsky <senozhatsky@chromium.org>
    Tested-by: Sergey Senozhatsky <senozhatsky@chromium.org>
    Link: https://lore.kernel.org/r/20220427185221.28928-1-peter.ujfalusi@linux.intel.com
    Signed-off-by: Mark Brown <broonie@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    sound/soc/sof/ipc3-topology.c | 39 +++++++++++++++++++++++------------
    1 file changed, 26 insertions(+), 13 deletions(-)

    diff --git a/sound/soc/sof/ipc3-topology.c b/sound/soc/sof/ipc3-topology.c
    index 2f8450a8c0a1..af1bbd34213c 100644
    --- a/sound/soc/sof/ipc3-topology.c
    +++ b/sound/soc/sof/ipc3-topology.c
    @@ -20,7 +20,8 @@
    struct sof_widget_data {
    int ctrl_type;
    int ipc_cmd;
    - struct sof_abi_hdr *pdata;
    + void *pdata;
    + size_t pdata_size;
    struct snd_sof_control *control;
    };

    @@ -784,16 +785,26 @@ static int sof_get_control_data(struct snd_soc_component *scomp,
    }

    cdata = wdata[i].control->ipc_control_data;
    - wdata[i].pdata = cdata->data;
    - if (!wdata[i].pdata)
    - return -EINVAL;

    - /* make sure data is valid - data can be updated at runtime */
    - if (widget->dobj.widget.kcontrol_type[i] == SND_SOC_TPLG_TYPE_BYTES &&
    - wdata[i].pdata->magic != SOF_ABI_MAGIC)
    - return -EINVAL;
    + if (widget->dobj.widget.kcontrol_type[i] == SND_SOC_TPLG_TYPE_BYTES) {
    + /* make sure data is valid - data can be updated at runtime */
    + if (cdata->data->magic != SOF_ABI_MAGIC)
    + return -EINVAL;
    +
    + wdata[i].pdata = cdata->data->data;
    + wdata[i].pdata_size = cdata->data->size;
    + } else {
    + /* points to the control data union */
    + wdata[i].pdata = cdata->chanv;
    + /*
    + * wdata[i].control->size is calculated with struct_size
    + * and includes the size of struct sof_ipc_ctrl_data
    + */
    + wdata[i].pdata_size = wdata[i].control->size -
    + sizeof(struct sof_ipc_ctrl_data);
    + }

    - *size += wdata[i].pdata->size;
    + *size += wdata[i].pdata_size;

    /* get data type */
    switch (cdata->cmd) {
    @@ -876,10 +887,12 @@ static int sof_process_load(struct snd_soc_component *scomp,
    */
    if (ipc_data_size) {
    for (i = 0; i < widget->num_kcontrols; i++) {
    - memcpy(&process->data[offset],
    - wdata[i].pdata->data,
    - wdata[i].pdata->size);
    - offset += wdata[i].pdata->size;
    + if (!wdata[i].pdata_size)
    + continue;
    +
    + memcpy(&process->data[offset], wdata[i].pdata,
    + wdata[i].pdata_size);
    + offset += wdata[i].pdata_size;
    }
    }

    --
    2.35.1


    \
     
     \ /
      Last update: 2022-06-08 00:18    [W:2.951 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site