Unnamed repository; edit this file 'description' to name the repository.
 help / color / mirror / Atom feed
* Re: [PATCH] media: venus: Fix possible buffer overflow in venus_sfr_print()
@ 2020-05-30  9:44 Markus Elfring
  0 siblings, 0 replies; only message in thread
From: Markus Elfring @ 2020-05-30  9:44 UTC (permalink / raw)
  To: Jia-Ju Bai, linux-arm-msm, linux-media
  Cc: kernel-janitors, linux-kernel, Andy Gross, Bjorn Andersson,
	Mauro Carvalho Chehab, Stanimir Varbanov

Please avoid typos in the patch subject.


> To fix this possible bug, sfr->buf_size is assigned to a local variable,
> and then this variable is checked before being used.

How do you think about a wording variant like the following?

  Thus assign the data structure member “buf_size” to a local variable
  and check it before further usage.


Would you like to add the tag “Fixes” to the commit message?

Regards,
Markus

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-05-30  9:44 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-30  9:44 [PATCH] media: venus: Fix possible buffer overflow in venus_sfr_print() Markus Elfring

Unnamed repository; edit this file 'description' to name the repository.

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://archive.lwn.net:8080/linux-media/0 linux-media/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-media linux-media/ http://archive.lwn.net:8080/linux-media \
		linux-media@vger.kernel.org lwn-linux-media@archive.lwn.net
	public-inbox-index linux-media

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://archive.lwn.net/lwn.kernel.linux-media


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git