Date: Fri, 1 Jan 2010 10:12:41 -0500 From: David Horn <dhorn2000@gmail.com> To: Manfred Usselmann <usselmann.m@icg-online.de> Cc: freebsd-stable@freebsd.org Subject: Re: duplicity ftp backup / ncftp no longer working since 8.0-Release Message-ID: <25ff90d61001010712r28f68ddeyd1fb1164be0c9091@mail.gmail.com> In-Reply-To: <20100101152827.8eabdf9e.usselmann.m@icg-online.de> References: <20091213103237.d01b51f2.usselmann.m@icg-online.de> <25ff90d60912132123x77198b1o6bfad3bffe0d01a0@mail.gmail.com> <20091214081716.f6e96b85.usselmann.m@icg-online.de> <20091214233123.a5b178c2.usselmann.m@icg-online.de> <25ff90d60912141451y50137b09qa2d5c380093dd8d5@mail.gmail.com> <20100101152827.8eabdf9e.usselmann.m@icg-online.de>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Jan 1, 2010 at 9:28 AM, Manfred Usselmann
<usselmann.m@icg-online.de> wrote:
> On Mon, 14 Dec 2009 17:51:31 -0500
> David Horn <dhorn2000@gmail.com> wrote:
>
>> On Mon, Dec 14, 2009 at 5:31 PM, Manfred Usselmann
>> <usselmann.m@icg-online.de> wrote:
>> > On Mon, 14 Dec 2009 08:17:16 +0100
>> > Manfred Usselmann <usselmann.m@icg-online.de> wrote:
>> >
>> >> On Mon, 14 Dec 2009 00:23:17 -0500
>> >> David Horn <dhorn2000@gmail.com> wrote:
>> >>
>> >> > I believe that there is something unusual going on with the checkin=
g
>> >> > on select() return in ncftp3. =A0If you change every instance of
>> >> > select() result checking in ftp/ncftp3 from "=3D=3D1" to ">=3D1" th=
e problem
>> >> > seems to go away.
>> >> >
>> >> > result =3D select(sfd + 1, NULL, SELECT_TYPE_ARG234 &ss, NULL,
>> >> > SELECT_TYPE_ARG5 &tv);
>> >> > -if (result =3D=3D 1) {
>> >> > +if (result >=3D 1) {
>> >>
>> >> I will try this.
>> >
>> > Did work for me!
>> >
>> > Thanks,
>> > Manfred
>>
>> OK. =A0 I will try to report it to the upstream (ncftp.com/contact), and
>> failing that we could always patch as part of the ncftp 3.2.3 update
>> into freebsd ports, but glad to hear it worked for you.
>
> The port has been updated to 3.2.3 but without a fix.
>
> http://www.freebsd.org/cgi/query-pr.cgi?pr=3D140934 mentions a fix to
> FreeBSD 8 which sounds like it should have solved our issue. Don't know
> why it didn't.
>
> Regards,
> Manfred
>
3.2.3 of ncftp was released by the vendor in July 2009
(http://www.ncftp.org/), so it would not have yet incorporated the
select() result checking changes.
I reported the issue to ncftp, and they acknowledged the issue. I can
only assume that it will be addressed in the next vendor release
(3.2.4 ?) The obrien change was a patch to the freebsd ncftp port
(3.2.2) for the same issue, but only on one of the select() instances
(not all of them).
http://www.freebsd.org/cgi/cvsweb.cgi/ports/ftp/ncftp3/files/Attic/patch-SC=
onnect.c
The obrien commit was since reverted for 3.2.3 since the vendor
included the one instance of select() result checking in the base
code.
So, long story short, you still need to manually patch the source of
ncftp until either
A) Vendor updates base code (or)
B) Someone submits a patch to the freebsd port of ftp/ncftp3 (3.2.3)
If you are really concerned about it, create an official patch (use
diff -u) and attach it to your bug report. Bug reports with patches
are much more likely to see action.
If you are having issues creating a patch let me know and I will try
to find the time to do this in the next week or so. (ENOTIME)
Good Luck.
---Dave Horn
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?25ff90d61001010712r28f68ddeyd1fb1164be0c9091>