Thread (2 messages) flat view 2 messages, 2 authors, 2016-02-08

Re: [PATCH] af_unix: Don't set err in unix_stream_read_generic unless there was an error

From: Sergei Shtylyov <hidden>
Date: 2016-02-08 18:33:46
Also in: lkml, stable

Possibly related (same subject, not in this thread)

Hello.

On 02/08/2016 06:33 PM, Rainer Weikusat wrote:
The present unix_stream_read_generic contains various code sequences of
the form

err = -EDISASTER;
if (<test>)
	goto out;

This has the unfortunate side effect of possibly causing the error code
to bleed through to the final

out:
	return copied ? : err;

and then to be wrongly returned if no data was copied because the caller
didn't supply a data buffer, as demonstrated by the program available at

http://pad.lv/1540731

Change it such that err is only set if an error condition was detected.

Fixes: 3822b5c2fc62
    You also need to specify the patch summary like this: ("<summary").
Signed-off-by: Rainer Weikusat <redacted>
[...]

MBR, Sergei
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help