Menu

#376 PDL Segmentation fault working with Storable restored PDL

feature_request
closed-fixed
None
1
2015-10-05
2015-03-16
No

Running the attached code generates a Segmentation fault.

$ ./test.pl
OS: Linux new 3.10.0-123.9.3.el7.x86_64 #1 SMP Thu Oct 30 00:16:40 EDT 2014 x86_64 x86_64 x86_64 GNU/Linux
Perl: v5.16.3
PDL: 2.007
Storable: 2.45

Segmentation fault

1 Attachments

Related

Bugs: #376

Discussion

  • Chris Marshall

    Chris Marshall - 2015-03-17

    It looks like the portability issues with Storable have been addressed
    since PDL-2.007.
    On Mar 16, 2015 1:41 PM, "Matthew McGillis" mmcgillis@users.sf.net wrote:


    Status: open
    Group: critical
    Created: Mon Mar 16, 2015 08:41 PM UTC by Matthew McGillis
    Last Updated: Mon Mar 16, 2015 08:41 PM UTC
    Owner: nobody

    Running the attached code generates a Segmentation fault.

    $ ./test.pl
    OS: Linux new 3.10.0-123.9.3.el7.x86_64 #1 SMP Thu Oct 30 00:16:40 EDT
    2014 x86_64 x86_64 x86_64 GNU/Linux
    Perl: v5.16.3
    PDL: 2.007
    Storable: 2.45

    Segmentation fault

    Sent from sourceforge.net because you indicated interest in
    https://sourceforge.net/p/pdl/bugs/376/

    To unsubscribe from further messages, please visit
    https://sourceforge.net/auth/subscriptions/

     

    Related

    Bugs: #376

  • Matthew McGillis

    Not exactly sure what you mean but this is my results with 2.007_12 which I just down loaded and built.

    $ ./test.pl
    OS: Linux new 3.10.0-123.9.3.el7.x86_64 #1 SMP Thu Oct 30 00:16:40 EDT 2014 x86_64 x86_64 x86_64 GNU/Linux
    Perl: v5.16.3
    PDL: 2.007_12
    Storable: 2.45

    Segmentation fault

     
  • Chris Marshall

    Chris Marshall - 2015-03-22

    It appears your test.pl is broken as far as PDL goes. See PDL::IO::Storable documentation but, basically, you left out 'use PDL::IO::Storable;' which makes things work for me.

     
  • Matthew McGillis

    Thanks for the pointer that does address the issue for me as well.

    Not sure how easy it is but adding something into the code to generate a warning or error about not having PDL::IO::Storable would be a lot friendlier than a Segmentation fault.

     
  • Chris Marshall

    Chris Marshall - 2015-03-24

    Good idea. I'll keep this ticket open as a reminder.

     
  • Chris Marshall

    Chris Marshall - 2015-03-24
    • status: open --> open-remind
    • Group: critical --> feature_request
    • Priority: 5 --> 1
     
  • Chris Marshall

    Chris Marshall - 2015-09-29

    I don't think this is an easy fix and don't think it will make PDL-2.014. One possibility would be to add PDL::IO::Storable to use PDL. Another might be to implement stub interface hooks for PDL that issue the warning if PDL::IO::Storable was not use-d. Seems like we might as well to option one in that case. Thoughts?

     

    Last edit: Chris Marshall 2015-10-02
  • Chris Marshall

    Chris Marshall - 2015-10-03

    PDL::IO::Storable is now loaded when "use PDL;"
    This isn't a perfect solution but does avoid mysterious SEGFAULT for many users.
    Fixed in git master to appear in next CPAN release.

     
  • Chris Marshall

    Chris Marshall - 2015-10-03
    • status: open-remind --> pending-fixed
    • assigned_to: Chris Marshall
     
  • Chris Marshall

    Chris Marshall - 2015-10-05

    With the comint PDL-2.014 release, PDL::IO::Storable will be loaded as part of "use PDL;" which should avoid the SEGFAULT for most cases.

     
  • Chris Marshall

    Chris Marshall - 2015-10-05
    • status: pending-fixed --> closed-fixed
     

Log in to post a comment.