Re: [tyndur-devel] [PATCH 1/2] ftp: fix behandlung von Servermeldungen mit Überlangen Zeilen

[Kevin Wolf <kevin@xxxxxxxxxx>]

On Mon, Apr 09, 2012 at 05:42:48PM +0200, Kevin Wolf wrote:
> On Mon, Apr 04, 2011 at 05:12:18PM +0200, Andreas Freimuth wrote:
> > ! ftp.c/response: kann jetzt auch mit Zeilen länger als LINE_LENGTH
> >   umgehen
> > 
> > Signed-off-by: Andreas Freimuth <m.nemo@xxxxxxx>
> > ---
> >  src/modules/c/ftp/ftp.c |   64 +++++++++++++++++++++-------------------------
> >  1 files changed, 29 insertions(+), 35 deletions(-)
> 
> Noch ein Uraltpatch, der bisher nicht committet worden ist... Auf den
> ersten Blick sieht er vernünftig aus, allerdings ist hier noch irgendwas
> kaputt. Wenn ich mich testweise auf ftp.debian.org einlogge und ein ls
> mache, gibt es rot.

Das Problem ist, dass der Patch nicht berücksichtigt, dass fgets() unter
tyndur nicht blockiert, bis es eine komplette Zeile gelesen hat, sondern
nur das zurückliefert, was schon angekommen ist.