Thread (4 messages) flat view 4 messages, 2 authors, 2017-05-09

Re: [PATCH net] tcp: init tcp_options before using it.

From: Eric Dumazet <hidden>
Date: 2017-05-08 14:49:27

On Mon, 2017-05-08 at 17:57 +0800, Hangbin Liu wrote:
I searched 4308fc58dced ("tcp: Document use of undefined variable") in
archive list, but did not find the thread. So I'm not sure why we only
add a description about un-initialized value.

Even we don't use tmp_opt.sack_ok, I think it would be more safe to
initialize the value before using it. Just as other caller did.
Patch is not needed at all.

Comment and code are pretty clear.

This part of the code uses a generic function ( tcp_parse_options()) to
decode TCP options, but we are only caring about TS one.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help