The cd->config could be passed as null pointer to eq_iir_setup() if the previous malloc() would fail.
Signed-off-by: Seppo Ingalsuo seppo.ingalsuo@linux.intel.com --- src/audio/eq_iir.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/audio/eq_iir.c b/src/audio/eq_iir.c index a8def32..6c8ddc6 100644 --- a/src/audio/eq_iir.c +++ b/src/audio/eq_iir.c @@ -339,9 +339,10 @@ static int eq_iir_cmd(struct comp_dev *dev, int cmd, void *data)
/* Allocate and make a copy of the blob and setup IIR */ cd->config = rmalloc(RZONE_RUNTIME, RFLAGS_NONE, bs); - if (cd->config != NULL) - memcpy(cd->config, blob->data, bs); + if (cd->config == NULL) + return -EINVAL;
+ memcpy(cd->config, blob->data, bs); eq_iir_setup(cd->iir, cd->config, source->params.pcm->channels); break; case COMP_CMD_MUTE: