Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7a01c26d authored by Prasad Kumpatla's avatar Prasad Kumpatla Committed by Gerrit - the friendly Code Review server
Browse files

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: default avatarPrasad Kumpatla <nkumpat@codeaurora.org>
parent 1038df1f

asoc/codecs/wcd937x/wcd937x.c

+8 −0
Original line number Diff line number Diff line
@@ -163,6 +163,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++) {
@@ -207,6 +211,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,