Subject: Re: SFTP issues

Re: SFTP issues

From: Peter Stuge <peter_at_stuge.se>
Date: Thu, 9 Feb 2012 13:27:43 +0100

Alexander Lamaison wrote:
> >> @@ -1348,13 +1334,13 @@ static ssize_t sftp_read(LIBSSH2_SFTP_HANDLE * handle, char *buffer,
> >>                      filep->offset_sent -= (chunk->len - rc32);
> >>                  }
> >>
> >> -                if(total_read + rc32 > buffer_size) {
> >> +                if(rc32 > buffer_size) {
> >>                      /* figure out the overlap amount */
> >> -                    filep->data_left = (total_read + rc32) - buffer_size;
> >> +                    filep->data_left = rc32 - buffer_size;
> >
> > Doesn't this lose the bytes that we received from transport but which
> > do not fit into the buffer given to us by the user?
>
> Not unless it did so before.

Sure. Look at the logic. data_left is being set to a smaller number
of bytes than actually received from transport. Where do the
remaining bytes go?

> > suggest return EAGAIN instead of having the loop and chunk
>
> I'll try this out tonight.

Cool.

//Peter
_______________________________________________
libssh2-devel http://cool.haxx.se/cgi-bin/mailman/listinfo/libssh2-devel
Received on 2012-02-09