Re: [PATCH] binder: fix null deref of proc->context

2020-06-22 Thread Greg Kroah-Hartman
On Mon, Jun 22, 2020 at 01:59:04PM -0700, Todd Kjos wrote: > On Mon, Jun 22, 2020 at 1:18 PM Todd Kjos wrote: > > > > On Mon, Jun 22, 2020 at 1:09 PM Christian Brauner > > wrote: > > > > > > On Mon, Jun 22, 2020 at 01:07:15PM -0700, Todd Kjos wrote: > > > > The binder driver makes the assumption

Re: [PATCH] binder: fix null deref of proc->context

2020-06-22 Thread Christian Brauner
On Mon, Jun 22, 2020 at 01:07:15PM -0700, Todd Kjos wrote: > The binder driver makes the assumption proc->context pointer is invariant > after > initialization (as documented in the kerneldoc header for struct proc). > However, in commit f0fe2c0f050d ("binder: prevent UAF for binderfs devices > I

Re: [bug report] staging: vchiq_arm: use list_for_each_entry when accessing bulk_waiter_list

2020-06-22 Thread Dan Carpenter
On Mon, Jun 22, 2020 at 04:56:19PM +0200, Nicolas Saenz Julienne wrote: > Hi Dan, > Thanks for reaching out. > > On Fri, 2020-06-19 at 17:31 +0300, Dan Carpenter wrote: > > Hello Nicolas Saenz Julienne, > > Feel free to refer to me as Nicolas, but if you'd rather keep it formal the > name is Nico

URGENT REPLY.

2020-06-22 Thread Karim Zakari
Good-Day Friend,  Hope you are doing great Today. I have a proposed business deal worthy  (US$16.5 Million Dollars) that will benefit both parties. This is legitimate'  legal and your personality will not be compromised. Waiting for your response for more details, As you are willing to execute 

Re: [bug report] staging: vchiq_arm: use list_for_each_entry when accessing bulk_waiter_list

2020-06-22 Thread Nicolas Saenz Julienne
Hi Dan, Thanks for reaching out. On Fri, 2020-06-19 at 17:31 +0300, Dan Carpenter wrote: > Hello Nicolas Saenz Julienne, Feel free to refer to me as Nicolas, but if you'd rather keep it formal the name is Nicolas Patricio Saenz Julienne. :P > > The patch 46e4b9ec4fa4: "staging: vchiq_arm: use l

[PATCH 0/6] staging: most: cdev: clean-up and improvements

2020-06-22 Thread Christian Gromm
Patch set to fix minor issues and improve things a little. Christian Gromm (6): staging: most: cdev: remove noisy log messages staging: most: cdev: use dev_*() functions to print messages staging: most: cdev: remove overcautious parameter checking staging: most: cdev: fix up parenthesizing

[PATCH 5/6] staging: most: cdev: fix return values

2020-06-22 Thread Christian Gromm
This patch makes use of the proper return values when reporting an error to the caller. Signed-off-by: Christian Gromm --- drivers/staging/most/cdev/cdev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/most/cdev/cdev.c b/drivers/staging/most/cdev/cdev.

[PATCH 1/6] staging: most: cdev: remove noisy log messages

2020-06-22 Thread Christian Gromm
This patch removes unnecessary log messages to avoid noise in the kernel log. Signed-off-by: Christian Gromm --- drivers/staging/most/cdev/cdev.c | 26 ++ 1 file changed, 6 insertions(+), 20 deletions(-) diff --git a/drivers/staging/most/cdev/cdev.c b/drivers/staging/mos

[PATCH 2/6] staging: most: cdev: use dev_*() functions to print messages

2020-06-22 Thread Christian Gromm
This patch removes the pr_*() functions and uses dev_*() instead. Signed-off-by: Christian Gromm --- drivers/staging/most/cdev/cdev.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/most/cdev/cdev.c b/drivers/staging/most/cdev/cdev.c index 093b44d

[PATCH 6/6] staging: most: cdev: simplify list iteration

2020-06-22 Thread Christian Gromm
This patch uses a less confusing list traversing structure to either return an item of the list that meets the prerequisites or NULL otherwise. Signed-off-by: Christian Gromm --- drivers/staging/most/cdev/cdev.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers

[PATCH 4/6] staging: most: cdev: fix up parenthesizing

2020-06-22 Thread Christian Gromm
This patch removes unnecessary parnthesis. Signed-off-by: Christian Gromm --- drivers/staging/most/cdev/cdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/most/cdev/cdev.c b/drivers/staging/most/cdev/cdev.c index 9fc84fa..d374cc0 100644 --- a/drivers/stagi

[PATCH 3/6] staging: most: cdev: remove overcautious parameter checking

2020-06-22 Thread Christian Gromm
The interface pointer passed to a component API function cannot be NULL. This patch removes unnecessary sanity checks of the pointer. Signed-off-by: Christian Gromm --- drivers/staging/most/cdev/cdev.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/staging/mos

Re: [PATCH v13 2/2] drm/bridge: anx7625: Add anx7625 MIPI DSI/DPI to DP

2020-06-22 Thread Xin Ji
On Sun, Jun 21, 2020 at 10:09:25AM +0200, Sam Ravnborg wrote: > Hi Xin. > > > On Tue, Jun 09, 2020 at 03:19:50PM +0800, Xin Ji wrote: > > The ANX7625 is an ultra-low power 4K Mobile HD Transmitter designed > > for portable device. It converts MIPI DSI/DPI to DisplayPort 1.3 4K. > > > > Signed-of

[PATCH v2] staging: gasket: replace symbolic permissions

2020-06-22 Thread Rodolfo C. Villordo
WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal permissions '0444'. + .attr = __ATTR(_name, S_IRUGO, _show_function, NULL), \ warning detected by checkpatch.pl Unable to use __ATTR_RO(). Driver has multiple files using the same show function:

Re: [PATCH] staging: gasket: replace symbolic permissions

2020-06-22 Thread Rodolfo C Villordo
On Fri, Jun 19, 2020 at 10:32:24AM +0200, Greg Kroah-Hartman wrote: > On Fri, Jun 19, 2020 at 08:27:14AM +, Rodolfo C Villordo wrote: > > On Thu, Jun 18, 2020 at 09:47:50AM +0200, Greg Kroah-Hartman wrote: > > > On Mon, Jun 01, 2020 at 12:52:40AM +, Rodolfo C. Villordo wrote: > > > > WARNIN