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

Commit dd8e5009 authored by qctecmdr's avatar qctecmdr Committed by Gerrit - the friendly Code Review server
Browse files

Merge "asoc: Add check for substream ref_count before access the substream."

parents a18ec3d3 ef02f940

asoc/msm-pcm-loopback-v2.c

+22 −12
Original line number Diff line number Diff line
@@ -552,6 +552,7 @@ static int msm_pcm_volume_ctl_put(struct snd_kcontrol *kcontrol, 
		goto exit;

	}

	mutex_lock(&loopback_session_lock);

	if (substream->ref_count > 0) {

		prtd = substream->runtime->private_data;

		if (!prtd) {

			rc = -ENODEV;
@@ -559,6 +560,7 @@ static int msm_pcm_volume_ctl_put(struct snd_kcontrol *kcontrol, 
			goto exit;

		}

		rc = pcm_loopback_set_volume(prtd, volume);

	}

	mutex_unlock(&loopback_session_lock);

exit:

	return rc;
@@ -584,6 +586,7 @@ static int msm_pcm_volume_ctl_get(struct snd_kcontrol *kcontrol, 
		goto exit;

	}

	mutex_lock(&loopback_session_lock);

	if (substream->ref_count > 0) {

		prtd = substream->runtime->private_data;

		if (!prtd) {

			rc = -ENODEV;
@@ -591,6 +594,7 @@ static int msm_pcm_volume_ctl_get(struct snd_kcontrol *kcontrol, 
			goto exit;

		}

		ucontrol->value.integer.value[0] = prtd->volume;

	}

	mutex_unlock(&loopback_session_lock);

exit:

	return rc;
@@ -888,6 +892,12 @@ static int msm_pcm_channel_mixer_cfg_ctl_put(struct snd_kcontrol *kcontrol, 
			chmixer_pspd);



	mutex_lock(&loopback_session_lock);

	if (substream->ref_count <= 0) {

		pr_err_ratelimited("%s: substream ref_count:%d invalid\n",

				__func__, substream->ref_count);

		mutex_unlock(&loopback_session_lock);

		return -EINVAL;

	}

	if (chmixer_pspd->enable && substream->runtime) {

		prtd = substream->runtime->private_data;

		if (!prtd) {

asoc/msm-pcm-q6-noirq.c

+11 −7
Original line number Diff line number Diff line
@@ -762,9 +762,11 @@ static int msm_pcm_volume_ctl_get(struct snd_kcontrol *kcontrol, 
		return -ENODEV;

	}

	mutex_lock(&pdata->lock);

	if (substream->ref_count > 0) {

		prtd = substream->runtime->private_data;

		if (prtd)

			ucontrol->value.integer.value[0] = prtd->volume;

	}

	mutex_unlock(&pdata->lock);

	return 0;

}
@@ -807,11 +809,13 @@ static int msm_pcm_volume_ctl_put(struct snd_kcontrol *kcontrol, 
	}



	mutex_lock(&pdata->lock);

	if (substream->ref_count > 0) {

		prtd = substream->runtime->private_data;

		if (prtd) {

			rc = msm_pcm_set_volume(prtd, volume);

			prtd->volume = volume;

		}

	}

	mutex_unlock(&pdata->lock);

	return rc;

}

asoc/msm-pcm-q6-v2.c

+47 −16
Original line number Diff line number Diff line
@@ -1330,7 +1330,12 @@ static int msm_pcm_adsp_stream_cmd_put(struct snd_kcontrol *kcontrol, 
		ret = -EINVAL;

		goto done;

	}



	if (substream->ref_count <= 0) {

		pr_err_ratelimited("%s substream ref_count:%d invalid\n",

				__func__, substream->ref_count);

		ret = -EINVAL;

		goto done;

	}

	prtd = substream->runtime->private_data;

	if (prtd == NULL) {

		pr_err("%s prtd is null.\n", __func__);
@@ -1561,9 +1566,11 @@ static int msm_pcm_volume_ctl_get(struct snd_kcontrol *kcontrol, 
	}



	mutex_lock(&pdata->lock);

	if (substream->ref_count > 0) {

		prtd = substream->runtime->private_data;

		if (prtd)

			ucontrol->value.integer.value[0] = prtd->volume;

	}

	mutex_unlock(&pdata->lock);

	return 0;

}
@@ -1607,11 +1614,13 @@ static int msm_pcm_volume_ctl_put(struct snd_kcontrol *kcontrol, 
	}



	mutex_lock(&pdata->lock);

	if (substream->ref_count > 0) {

		prtd = substream->runtime->private_data;

		if (prtd) {

			rc = msm_pcm_set_volume(prtd, volume);

			prtd->volume = volume;

		}

	}

	mutex_unlock(&pdata->lock);

	return rc;

}
@@ -1678,9 +1687,11 @@ static int msm_pcm_compress_ctl_get(struct snd_kcontrol *kcontrol, 
		return 0;

	}

	mutex_lock(&pdata->lock);

	if (substream->ref_count > 0) {

		prtd = substream->runtime->private_data;

		if (prtd)

			ucontrol->value.integer.value[0] = prtd->compress_enable;

	}

	mutex_unlock(&pdata->lock);

	return 0;

}
@@ -1710,12 +1721,14 @@ static int msm_pcm_compress_ctl_put(struct snd_kcontrol *kcontrol, 
		return 0;

	}

	mutex_lock(&pdata->lock);

	if (substream->ref_count > 0) {

		prtd = substream->runtime->private_data;

		if (prtd) {

			pr_debug("%s: setting compress flag to 0x%x\n",

			__func__, compress);

			prtd->compress_enable = compress;

		}

	}

	mutex_unlock(&pdata->lock);

	return rc;

}
@@ -1824,6 +1837,12 @@ static int msm_pcm_chmap_ctl_put(struct snd_kcontrol *kcontrol, 
		return 0;



	mutex_lock(&pdata->lock);

	if (substream->ref_count <= 0) {

		pr_err_ratelimited("%s: substream ref_count:%d invalid\n",

				__func__, substream->ref_count);

		mutex_unlock(&pdata->lock);

		return -EINVAL;

	}

	prtd = substream->runtime ? substream->runtime->private_data : NULL;

	if (prtd) {

		prtd->set_channel_map = true;
@@ -1891,6 +1910,12 @@ static int msm_pcm_chmap_ctl_get(struct snd_kcontrol *kcontrol, 
		return 0; /* no channels set */



	mutex_lock(&pdata->lock);

	if (substream->ref_count <= 0) {

		pr_err_ratelimited("%s: substream ref_count:%d invalid\n",

				__func__, substream->ref_count);

		mutex_unlock(&pdata->lock);

		return -EINVAL;

	}

	prtd = substream->runtime ? substream->runtime->private_data : NULL;



	if (prtd && prtd->set_channel_map == true) {
@@ -2180,6 +2205,12 @@ static int msm_pcm_channel_mixer_cfg_ctl_put(struct snd_kcontrol *kcontrol, 
	}



	mutex_lock(&pdata->lock);

	if (substream->ref_count <= 0) {

		pr_err_ratelimited("%s: substream ref_count:%d invalid\n",

				__func__, substream->ref_count);

		mutex_unlock(&pdata->lock);

		return -EINVAL;

	}

	prtd = substream->runtime ? substream->runtime->private_data : NULL;

	if (chmixer_pspd->enable && prtd) {

		if (session_type == SESSION_TYPE_RX &&