Re: [perl #43105] [PATCH] function mmd.c::mmd_expand_y: newly allocated memory unitialized

2007-06-05 Thread chromatic
On Monday 04 June 2007 11:10:01 Bernhard Schmalhofer via RT wrote: > This answers the question. So I propose to apply Mehmet's patch and > add to pdd07 something like: Thanks, both Mehmet's and your patches applied in r18832. -- c

[perl #43105] [PATCH] function mmd.c::mmd_expand_y: newly allocated memory unitialized

2007-06-04 Thread Bernhard Schmalhofer via RT
> > What happens when NULL is not a consective series of '\0' chars? > > I think that it breaks. > > > Are there such platforms and are they releavant for parrot? > > I believe that this assumption is endemic in Perl 5, and it's never > hindered > Perl 5's portability. The C FAQ gives examples

Re: [perl #43105] [PATCH] function mmd.c::mmd_expand_y: newly allocated memory unitialized

2007-06-04 Thread Bernhard Schmalhofer
Nicholas Clark schrieb: On Sun, Jun 03, 2007 at 08:00:18AM -0700, Bernhard Schmalhofer via RT wrote: I have looked at the 'more_memory.patch' and I'm wondering about the portability. In that patch loops where pointers are explicitly set to NULL are replaced with a memset( start, 0, len);

Re: [perl #43105] [PATCH] function mmd.c::mmd_expand_y: newly allocated memory unitialized

2007-06-03 Thread Nicholas Clark
On Sun, Jun 03, 2007 at 08:00:18AM -0700, Bernhard Schmalhofer via RT wrote: > I have looked at the 'more_memory.patch' and I'm wondering about the > portability. > In that patch loops where pointers are explicitly set to NULL > are replaced with a > memset( start, 0, len); > > What happens w

[perl #43105] [PATCH] function mmd.c::mmd_expand_y: newly allocated memory unitialized

2007-06-03 Thread Bernhard Schmalhofer via RT
> I grepped for other files that can make use of mem_sys_*_zeroed > variants. I attached a patch that affects objects.c, vtables.c, > gc/register.c and stm/waitlist.c. I have looked at the 'more_memory.patch' and I'm wondering about the portability. In that patch loops where pointers are explicit

Re: [perl #43105] [PATCH] function mmd.c::mmd_expand_y: newly allocated memory unitialized

2007-06-02 Thread Mehmet Yavuz Selim Soyturk
On 6/2/07, chromatic <[EMAIL PROTECTED]> wrote: On Friday 01 June 2007 16:29:08 Mehmet Yavuz Selim Soyturk wrote: > After a suggestion of Bernhard at #parrot, I added a function named > C to parrot: > > void * mem__sys_realloc_zeroed(void *from, size_t size, size_t > old_size) > > It zeroes t

Re: [perl #43105] [PATCH] function mmd.c::mmd_expand_y: newly allocated memory unitialized

2007-06-01 Thread chromatic
On Friday 01 June 2007 13:39:32 Mehmet Yavuz Selim Soyturk wrote: > Function mmd_expand_y in src/mmd.c allocates new memory for the mmd > table, but does not initialize the newy allocated memory to NULL, > which causes segfaults for some cases. The attached patch solves that > problem. I thought

[perl #43105] [PATCH] function mmd.c::mmd_expand_y: newly allocated memory unitialized

2007-06-01 Thread Mehmet Yavuz Selim Soyturk
# New Ticket Created by "Mehmet Yavuz Selim Soyturk" # Please include the string: [perl #43105] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=43105 > Function mmd_expand_y in src/mmd.c allocates new memory for the mmd tabl