Ticket #107 (closed defect)

Opened 8 years ago

Last modified 8 years ago

send_exiting: debugdump wrong location

Reported by: anonymous Owned by: bagder
Priority: normal Milestone:
Component: misc Version:
Keywords: Cc: bagder
Blocked By: Blocks:

Description

In transport.c:send_existing() is the following statement

debugdump(session, "libssh2_transport_write send()", &p->outbuf[p->osent],

length);

at the wrong location. If debugging is enabled debugdump() may try to dump the memory at p->outbuf which is set to NULL in case the remainder of the package was sent. This cause a SigSegV.

Solution: move the statement before the "if (rc == length)"-block.

Change History

comment:1 Changed 8 years ago by bagder

Thanks for your report and help in improving libssh2!

A fix has just been committed to the source code repo, this case is now considered fixed and closed!

Note: See TracTickets for help on using tickets.