[Bug 2804] channels.c:3258: suspicious code ?

[bugzilla-daemon at bugzilla.mindrot.org]

https://bugzilla.mindrot.org/show_bug.cgi?id=2804

Jakub Jelen <jjelen at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CLOSED                      |REOPENED
         Resolution|WORKSFORME                  |---
                 CC|                            |jjelen at redhat.com

--- Comment #4 from Jakub Jelen <jjelen at redhat.com> ---
Damien,
could we simplify this code as explained in the comment #2 and possibly
add a comment so it is clear for future readers?

This popped up in results of our static analyzers scanner as a
suspicious code again.

https://github.com/openssh/openssh-portable/blob/master/channels.c#L3356

-- 
You are receiving this mail because:
You are watching the assignee of the bug.
You are watching someone on the CC list of the bug.