SPECS: rsync.spec - drop out -ipv6_socket.patch introduced by plut...

Jakub Bogusz qboosh w pld-linux.org
Śro, 25 Sie 2004, 08:28:55 CEST


On Wed, Aug 25, 2004 at 02:59:02AM +0200, Tomasz Wittner wrote:
> On Wed 25. of August 2004 02:28, havner wrote:
> > On Wednesday 25 August 2004 02:19, Tomasz Wittner wrote:
> > > Czekałem od maja.
> > > http://cvs.pld-linux.org/cgi-bin/cvsweb/SPECS/rsync.spec?r1=1.80&r2=1.81
> >
> > Ok, nie wiedzialem, ze tak dlugo, poszlo do pieca. A pluta zaraz pogonie.
> #ifdef IPV6_V6ONLY
>                   if (resp->ai_family == AF_INET6) {
>                       if(setsockopt(s, IPPROTO_IPV6, IPV6_V6ONLY, (char 
> *)&one, sizeof one) < 0)
>                           continue;
>                   }
> #endif
> Całą awantura jest oczywiście o nic - zmiana jest o ifdefowana, IPV6_V6ONLY 
> nie jest nigdzie ustawiane

Jest:
$ grep IPV6_V6ONLY -r /usr/include/bits/
/usr/include/bits/in.h:#define IPV6_V6ONLY              26

> (stąd ./configure needs fixes - trzeba dorobić do 
> niego switcha). Można w tym miejscu nasrać i nic się nie stanie - lepiej 
> jednak wylewać tony literek, niż zajrzeć do kodu. Pewnie pluto coś 
> kombinował, żeby mieć rsynca ipv6-only

A to trzeba by się przyjrzeć jak to się zachowuje na jądrach 2.4
(nie obsługujących IPV6_V6ONLY). Tam nie uda się bind do tego samego
portu jednocześnie po v4 i v6.

>  - - skipping the IPV6 socket, było 
> jednak "a bit confused". Swoją drogą, źle, że zostawił niepełne rel. - 
> "bierzemy z HEAD" i powinno być ono w takim razie "czyste", własnie na 
> wypadek konieczności dodawania sec. fixów.

Poprawka i tak wadliwa, bo:
- zawsze pomija adresy IPv6 na jądrach 2.4
- zostawia nie zamknięte, nie używane później gniazda


-- 
Jakub Bogusz    http://cyber.cs.net.pl/~qboosh/




Więcej informacji o liście dyskusyjnej pld-devel-pl