Bug #67136 for NET-Sieve: sfinish/DESTROY

Fri Apr 01 03:15:29 2011 kd [...] tu-cottbus.de - Ticket created

Subject:	sfinish/DESTROY
Date:	Fri, 01 Apr 2011 09:15:14 +0200
To:	bug-net-sieve [...] rt.cpan.org
From:	Klaus-Dieter Krannich <kd [...] tu-cottbus.de>

Hi, Net::Sieve is a very useful piece of Software, thank you. However, in the sfinish function the two calls to ssend have a wrong argument list, should be $self->ssend($_[0]) (instaed of $self->ssend($sock, $_[0])) and $self->ssend("LOGOUT") (instead of $self->ssend($sock, "LOGOUT")). In DESTROY $self->sfinish() should be called only, if a connection exists, ( $self->sfinish() if $self->sock; ? ). (use of undefined value e.g after "connection timeout") Best regards Klaus-Dieter Krannich ---------------------------------------------------------------- This message was sent using IMP, the Internet Messaging Program.

Download smime.p7s
application/pkcs7-signature 5.4k

Message body not shown because it is not plain text.

Fri Apr 01 03:43:03 2011 agostini [...] univ-metz.fr - Taken

Fri Apr 01 03:43:09 2011 agostini [...] univ-metz.fr - Status changed from 'new' to 'open'

Fri Apr 01 05:23:06 2011 agostini [...] univ-metz.fr - Correspondence added

Le Ven 01 Avr 2011 03:15:29, kd@tu-cottbus.de a écrit : Show quoted text

> Hi, > > Net::Sieve is a very useful piece of Software, thank you. > However, in the sfinish function the two calls to ssend have a wrong > argument list, should be $self->ssend($_[0]) (instaed of > $self->ssend($sock, $_[0])) > and $self->ssend("LOGOUT") (instead of $self->ssend($sock, "LOGOUT")). > In DESTROY $self->sfinish() should be called only, if a connection exists, > ( $self->sfinish() if $self->sock; ? ). (use of undefined value e.g > after "connection timeout") > > Best regards > > Klaus-Dieter Krannich > > ---------------------------------------------------------------- > This message was sent using IMP, the Internet Messaging Program. >

thanks for your bug report this is fixed in 0.07, currently uploaded on cpan Best regards Yves

Fri Apr 01 10:11:53 2011 agostini [...] univ-metz.fr - Status changed from 'open' to 'resolved'

Fri Apr 01 10:11:53 2011 agostini [...] univ-metz.fr - Severity Important added

Fri Apr 01 10:11:53 2011 agostini [...] univ-metz.fr - Fixed in 0.07 added

Fri Apr 01 10:15:59 2011 kd [...] tu-cottbus.de - Correspondence added

Subject:	Re: [rt.cpan.org #67136] sfinish/DESTROY
Date:	Fri, 01 Apr 2011 16:15:44 +0200
To:	bug-NET-Sieve [...] rt.cpan.org
From:	Klaus-Dieter Krannich <kd [...] tu-cottbus.de>

thank you for this quick response Regards Klaus-Dieter Show quoted text

> > > thanks for your bug report > > this is fixed in 0.07, currently uploaded on cpan > > Best regards > > Yves > > >

----- Ende der Nachricht von bug-NET-Sieve@rt.cpan.org ----- ---------------------------------------------------------------- This message was sent using IMP, the Internet Messaging Program.

Download smime.p7s
application/pkcs7-signature 5.4k

Message body not shown because it is not plain text.

Fri Apr 01 10:16:00 2011 The RT System itself - Status changed from 'resolved' to 'open'

Mon Apr 04 11:32:06 2011 agostini [...] univ-metz.fr - Status changed from 'open' to 'resolved'