From 924c7d027c76044a94d6148e9a38b8f0c93f9dd9 Mon Sep 17 00:00:00 2001 From: Prasad Kumpatla Date: Thu, 9 Apr 2020 17:53:35 +0530 Subject: [PATCH] asoc: codecs: Add default case for switch. Add default case for switch to avoid the Null pointer dereference for the map and num_ports variable in for loop. Change-Id: I59c9dc6e5396bcb409876378c811143da1c81551 Signed-off-by: Prasad Kumpatla --- asoc/codecs/wcd937x/wcd937x.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/asoc/codecs/wcd937x/wcd937x.c b/asoc/codecs/wcd937x/wcd937x.c index 314d50d6ce..2e3d27df0f 100644 --- a/asoc/codecs/wcd937x/wcd937x.c +++ b/asoc/codecs/wcd937x/wcd937x.c @@ -164,6 +164,10 @@ static int wcd937x_set_port_params(struct snd_soc_component *component, map = &wcd937x->tx_port_mapping; num_ports = wcd937x->num_tx_ports; break; + default: + dev_err(component->dev, "%s Invalid path selected %u\n", + __func__, path); + return -EINVAL; } for (i = 0; i <= num_ports; i++) { @@ -208,6 +212,10 @@ static int wcd937x_parse_port_mapping(struct device *dev, map = &wcd937x->tx_port_mapping; num_ports = &wcd937x->num_tx_ports; break; + default: + dev_err(dev, "%s Invalid path selected %u\n", + __func__, path); + return -EINVAL; } if (!of_find_property(dev->of_node, prop,