From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1hfMj8-0003Fl-8M for ath11k@lists.infradead.org; Mon, 24 Jun 2019 11:01:35 +0000 From: Kalle Valo Subject: Re: [PATCH] ath11k: fix invalid memory access References: <1561056973-32524-1-git-send-email-periyasa@codeaurora.org> <20190624105458.025FF609D4@smtp.codeaurora.org> Date: Mon, 24 Jun 2019 14:01:30 +0300 In-Reply-To: <20190624105458.025FF609D4@smtp.codeaurora.org> (Kalle Valo's message of "Mon, 24 Jun 2019 10:54:58 +0000 (UTC)") Message-ID: <87h88fw8l1.fsf@kamboji.qca.qualcomm.com> MIME-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "ath11k" Errors-To: ath11k-bounces+kvalo=adurom.com@lists.infradead.org To: Karthikeyan Periyasamy Cc: ath11k@lists.infradead.org Kalle Valo writes: > Karthikeyan Periyasamy wrote: > >> dp_reo_cache_flush_elem element get dynamically allocated with invalid size >> which leads to the access of unallocated memory region or memory corruption. >> Fix this bug by allocate with dp_reo_cache_flush_elem structure size. >> >> Signed-off-by: Karthikeyan Periyasamy >> Signed-off-by: Kalle Valo > > 11 patches applied to ath11k-bringup branch of ath.git, thanks. > > f1df70c948f4 ath11k: fix invalid memory access > 3213345e51fc ath11k: Remove unnecessary whitespaces > 9d18a5b1d777 ath11k: Avoid blank lines related warning > e3b79f9ec2a4 ath11k: Avoid space related warning > dcc1fd37c6dd ath11k: Fix Alignment should match open parenthesis warning > d2b20789808d ath11k: Fix misspelled warnings > e4e738f422d4 ath11k: fix unnecessary brace warnings > 61bfed886e93 ath11k: fix Block comments warnings > 1e9c09edaba3 ath11k: fix SPDX comment style warnings > 6a82b2e7f17e ath11k: fix return statements on void function warnings > ff1effad0d9b ath11k: fix string split across warnings BTW, you could have put all these checkpatch cleanup patches into one patchset (ie. one patcheset with 11 patches). Easier to manage them that way. If the patches are somehow related it's usually a good idea to combine them to the same patchset. -- Kalle Valo _______________________________________________ ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k