Bug#279323: Fix for Zope segfault on amd64

2005-01-24 Thread Per Bojsen
*** Regarding Re: Bug#279323: Fix for Zope segfault on amd64; [EMAIL PROTECTED] (A Mennucc) adds: A> Does zope2.7 segfauls as well? I don't know, I haven't tried it. A> In zope2.7 the line is A> if (!PyArg_ParseTuple(args, "sI:initgroups", &username,

Bug#279323: Fix for Zope segfault on amd64

2005-01-24 Thread A Mennucc
hi everybody Does zope2.7 segfauls as well? In zope2.7 the line is if (!PyArg_ParseTuple(args, "sI:initgroups", &username, &igid)) From http://www.python.org/doc/2.4/api/arg-parsing.html "I" (integer) [unsigned int] Convert a Python integer to a C unsigned int, without overflow c

Bug#279323: Fix for Zope segfault on amd64

2005-01-22 Thread Goswin von Brederlow
Per Bojsen <[EMAIL PROTECTED]> writes: > *** Regarding Re: Fix for Zope segfault on amd64; Vincent Hanquez > <[EMAIL PROTECTED]> adds: > > Vincent> Please be specific then. point one architecture that zope > Vincent> support which did not fulfill this requirement. > > There are versions of UNI

Bug#279323: Fix for Zope segfault on amd64

2005-01-22 Thread Per Bojsen
*** Regarding Re: Fix for Zope segfault on amd64; Vincent Hanquez <[EMAIL PROTECTED]> adds: Vincent> Please be specific then. point one architecture that zope Vincent> support which did not fulfill this requirement. There are versions of UNIX where gid_t is an unsigned short. Whether those a

Bug#279323: Fix for Zope segfault on amd64

2005-01-22 Thread Vincent Hanquez
On Sat, Jan 22, 2005 at 04:09:41PM -0500, Per Bojsen wrote: > This may be true for Linux but I wouldn't be so certain this is true > for all platforms Zope runs on. Please be specific then. point one architecture that zope support which did not fulfill this requirement. > You're making the assump

Bug#279323: Fix for Zope segfault on amd64

2005-01-22 Thread Per Bojsen
*** Regarding Re: Fix for Zope segfault on amd64; Vincent Hanquez <[EMAIL PROTECTED]> adds: Vincent> This is wrong. gid_t is 32 bits on all architectures. This may be true for Linux but I wouldn't be so certain this is true for all platforms Zope runs on. Vincent> You should use "si:initgrou

Bug#279323: Fix for Zope segfault on amd64

2005-01-22 Thread Vincent Hanquez
On Sat, Jan 22, 2005 at 09:49:50AM -0500, Per Bojsen wrote: > --- /tmp/Zope-2.6.4-src/lib/Components/initgroups/initgroups.c 2002-07-25 > 17:54:02.0 -0400 > +++ lib/Components/initgroups/initgroups.c 2005-01-22 09:41:56.882300943 > -0500 > @@ -23,12 +23,15 @@ > initgroups_initgroup

Bug#279323: Fix for Zope segfault on amd64

2005-01-22 Thread Per Bojsen
*** Regarding Re: Fix for Zope segfault on amd64; Goswin von Brederlow <[EMAIL PROTECTED]> adds: Goswin> Shouldn't that bug appear on all 64bit architectures and the Goswin> patch work on every architecture, even 32bit. Int is 32bit on Goswin> all debian archs while long varies. Agreed. Sinc

Bug#279323: Fix for Zope segfault on amd64

2005-01-22 Thread Goswin von Brederlow
Per Bojsen <[EMAIL PROTECTED]> writes: > Hi, > > I did some debugging and found out that the segfault in Zope on amd64 > is due to some non-64-bit-compatible code in the initgroups.c source > file (lib/Components/initgroups/initgroups.c in the Zope source > package). Here is a patch that fixes th

Bug#279323: Fix for Zope segfault on amd64

2005-01-21 Thread Per Bojsen
Hi, I did some debugging and found out that the segfault in Zope on amd64 is due to some non-64-bit-compatible code in the initgroups.c source file (lib/Components/initgroups/initgroups.c in the Zope source package). Here is a patch that fixes this problem: --- initgroups.c~ 2002-07-25 17: