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

Commit 5cfc6d22 authored by Mohammed Nayeem Ur Rahman's avatar Mohammed Nayeem Ur Rahman
Browse files

msm: adsprpc : Change to resolve undefined behaviour 



Resolving the undefined behaviour scenario where if adsp proc
init fails return error and not to goto bail to deinitialize
memory and unlock mutex.

Change-Id: Idf107aa340a211edec9121249e090c3e7c0e2200
Acked-by: default avatarGururaj Chalger <gchalger@qti.qualcomm.com>
Signed-off-by: default avatarMohammed Nayeem Ur Rahman <mohara@codeaurora.org>
parent 5bd24ffb

drivers/char/adsprpc.c

+1 −1
Original line number Diff line number Diff line
@@ -3626,7 +3626,7 @@ static int fastrpc_internal_munmap(struct fastrpc_file *fl, 
		pr_err("adsprpc: ERROR: %s: user application %s trying to unmap without initialization\n",

			 __func__, current->comm);

		err = EBADR;

		goto bail;

		return err;

	}

	mutex_lock(&fl->internal_map_mutex);