Re: [HACKERS] inet data type regression test fails - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [HACKERS] inet data type regression test fails
Date
Msg-id 199905091456.KAA22228@candle.pha.pa.us
Whole thread Raw
In response to inet data type regression test fails  (Tatsuo Ishii <t-ishii@sra.co.jp>)
Responses Re: [HACKERS] inet data type regression test fails  (Taral <taral@taral.net>)
List pgsql-hackers
Can someone comment on this one?  Is it fixed?


> Hi all,
> 
> The inet regression test has been failed on my LinuxPPC. While
> investigating the reason, I found a code that doesn't work on
> LinuxPPC. From network_broadcast() in utils/adt/network.c:
> 
> int    addr = htonl(ntohl(ip_v4addr(ip)) | (0xffffffff >> ip_bits(ip)));
> 
> Here ip_bits() returns from (unsigned char)0 to 32. My question is:
> what is the correct result of (0xffffffff >> ip_bits())?
> 
> 1. 0x0
> 2. 0xffffffff (actually does nothing)
> 
> LinuxPPC is 1. FreeBSD and Solaris are 2. network_broadcast() seems to
> expect 2. My guess is shifting over 32bit against a 32bit integer is
> not permitted and the result is platform depedent. If this would true,
> it could be said that network_broadcast() has a portabilty
> problem. Comments?
> ---
> Tatsuo Ishii
> 
> 


--  Bruce Momjian                        |  http://www.op.net/~candle maillist@candle.pha.pa.us            |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026
 


pgsql-hackers by date:

Previous
From: Vadim Mikheev
Date:
Subject: Re: [HACKERS] MVCC vacuum error
Next
From: Bruce Momjian
Date:
Subject: Re: [HACKERS] Problem with complex query