[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: Perl interface: SVN::Core::dirent_canonicalize segfaults on undef

From: Bert Huijben <bert_at_qqmail.nl>
Date: Fri, 22 Jan 2016 14:34:45 +0100

> -----Original Message-----
> From: Vincent Lefevre [mailto:vincent-svn_at_vinc17.net]
> Sent: vrijdag 22 januari 2016 13:48
> To: dev_at_subversion.apache.org
> Subject: Perl interface: SVN::Core::dirent_canonicalize segfaults on undef
>
> I have reported the following Debian bug:
>
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=812318
>
> With Subversion 1.9.3:
>
> $ perl -MSVN::Core -e 'SVN::Core::dirent_canonicalize(undef)'
> zsh: segmentation fault (core dumped) perl -MSVN::Core -e
> 'SVN::Core::dirent_canonicalize(undef)'
>
> In case of undef, I suppose that the Perl module should pass the empty
> string to the library (for which there are no errors), or return some
> other kind of error.
>
> I have not tried other kinds of arguments, but in any case, the
> module should make sure that a valid string is passed to the
> library.

The current perl wrapper is almost completely generated by swig, directly
from the Subversion C library. Just like our current python and ruby
wrappers.

This makes it easy to maintain (even without active maintainers for the
specific code), but also adds extra requirements... those of the C library
itself.

Feel free to report bugs, but I don't think we currently have an active
maintainer to solve them in a way you would like.

A somewhat higher level library around these apis would help a lot of users
as it hides these requirements and makes things much easier to use, but it
takes someone to maintain such a thing.

(I did exactly this for C#/.Net in the SharpSvn project)

        Bert
Received on 2016-01-22 14:35:02 CET

This is an archived mail posted to the Subversion Dev mailing list.