[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



On Mon, Apr 09, 2012 at 06:11:27PM +0200, Kevin Wolf wrote:
> 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.

Entweder tut es das jetzt nicht mehr oder ich habe das Problem nur
zufällig nicht mehr reproduzieren können. Jedenfalls habe ich die
Patchserie jetzt einfach mal reingenommen. Hat ja auch lang genug
gewartet.

Kevin