Skip Menu |

This queue is for tickets about the Set-Object CPAN distribution.

Report information
The Basics
Id: 53182
Status: resolved
Priority: 0/
Queue: Set-Object
People
Owner: Nobody in particular
Requestors: ais523 [...] bham.ac.uk
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.27
Fixed in: (no value)

History Show all quoted text
  Mon Dec 28 07:59:07 2009 ais523 [...] bham.ac.uk - Ticket created
Subject: Adding unblessed references to a weak set causes a segfault

The following program causes Perl to crash with a segmentation fault, no other error message displayed:

use Set::Object qw/weak_set/;

my $n = 1;
my $a = \$n;
my $set1 = weak_set();
$set1->insert($a);

The defining factor seems to be that the object that $a refers to is unblessed when added to the set. Expected behaviour is for the object to be added to the set without a segfault.

Tested with Set::Object 1.27, perl version 5.10.0, on Linux (2.6.31-17, packaged in Ubuntu Karmic).

  Wed Feb 13 07:13:40 2013 SAMV [...] cpan.org - Correspondence added
I am not accepting bug reports without a test case, sorry.
  Wed Feb 13 07:13:41 2013 The RT System itself - Status changed from 'new' to 'open'
  Wed Feb 13 07:13:42 2013 SAMV [...] cpan.org - Status changed from 'open' to 'rejected'
  Wed Feb 13 07:51:18 2013 NUFFIN [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #53182] Adding unblessed references to a weak set causes a segfault
Date: Wed, 13 Feb 2013 14:50:46 +0200
To: bug-Set-Object [...] rt.cpan.org
From: Yuval Kogman <nuffin [...] cpan.org>
The example code produces a segfault for me, all that's needed to make it into a test case is a plan + a pass()...
  Wed Feb 13 07:51:20 2013 The RT System itself - Status changed from 'rejected' to 'open'
  Wed Feb 13 10:10:21 2013 SAMV [...] cpan.org - Correspondence added
RT-Send-CC: nuffin [...] cpan.org
Hmm good point Yuval. I've added the test, but it seems to be fixed already: probably after the patch from the other RT ticket which supplied an actual TAP test script and patch was applied. So I'll close this for now.
  Wed Feb 13 10:10:24 2013 SAMV [...] cpan.org - Status changed from 'open' to 'resolved'
  Wed Feb 13 10:11:23 2013 SAMV [...] cpan.org - Correspondence added
The test script was added as https://github.com/samv/Set- Object/commit/87db8556377dc61aeafbf82e4862b94549cd4cb9
  Wed Feb 13 10:11:24 2013 The RT System itself - Status changed from 'resolved' to 'open'
  Wed Feb 13 10:12:19 2013 SAMV [...] cpan.org - Status changed from 'open' to 'resolved'