Skip to Content.
Sympa Menu

devel - [Daniel Pittman <daniel@rimspace.net>, Daniel Pittman <daniel@rimspace.net>] Bug#143081: wwsympa: Wrong test makes *all* alternate addresses invalid.

Subject: Developers of Sympa

List archive

Chronological Thread  
  • From: address@concealed (Jérôme Marant)
  • To: address@concealed
  • Subject: [Daniel Pittman <address@concealed>, Daniel Pittman <address@concealed>] Bug#143081: wwsympa: Wrong test makes *all* alternate addresses invalid.
  • Date: Tue, 16 Apr 2002 07:54:15 +0200


Hi,

I'm forwarding the bug report.

Cheers,

--- Begin Message ---
  • From: Daniel Pittman <address@concealed>
  • From: Daniel Pittman <address@concealed>
  • To: Debian Bug Tracking System <address@concealed>
  • Subject: Bug#143081: wwsympa: Wrong test makes *all* alternate addresses invalid.
  • Date: Tue, 16 Apr 2002 10:40:22 +1000
  • Resent-cc: Jerome Marant <address@concealed>, address@concealed
  • Resent-date: Tue, 16 Apr 2002 00:48:01 GMT
  • Resent-from: Daniel Pittman <address@concealed>
  • Resent-to: address@concealed
Package: wwsympa
Version: 3.3.3-2
Severity: normal

There is a bug in wwsympa.fcgi, in the alternate email address code,
that makes all email addresses invalid. It's a simple 'if'/'unless'
transposition.

This patch correctly the problem. Without it the package will not work
correctly, refusing any email address (except one that is eq the
current) as invalid.

Daniel

--- wwsympa.fcgi.orig Tue Apr 16 10:36:28 2002
+++ /usr/lib/cgi-bin/sympa/wwsympa.fcgi Tue Apr 16 10:36:34 2002
@@ -1510,7 +1510,7 @@
}

## Alt email is the same as main email address
- unless ($in{'new_alternative_email'} eq $param->{'user'}{'email'}) {
+ if ($in{'new_alternative_email'} eq $param->{'user'}{'email'}) {
&error_message('incorrect_email', {'email' =>
$in{'new_alternative_email'}});
&do_log('notice', "do_record_email:incorrect email
%s",$in{'new_alternative_email'});
return 'pref';


-- System Information
Debian Release: 3.0
Kernel Version: Linux inanna 2.5.6 #3 Mon Mar 11 23:34:33 EST 2002 i686
unknown

Versions of the packages wwsympa depends on:
ii apache 1.3.24-2 Versatile, high-performance HTTP server
ii debconf 1.0.32 Debian configuration management system
ii fileutils 4.1-10 GNU file management utilities
ii libcgi-fast-pe 5.6.1-7 CGI::Fast Perl module.
ii libfcgi-perl 0.64-1 FastCGI Perl module
ii mhonarc 2.5.2-1 Mail to HTML converter
ii perl-suid 5.6.1-7 Runs setuid Perl scripts.
ii sympa 3.3.3-2 Modern mailing list manager
ii apache 1.3.24-2 Versatile, high-performance HTTP server
^^^ (Provides virtual package httpd)

--
It is the final proof of God's omnipotence that he need not exist in order to
save us.
-- Peter De Vries


--- End Message ---


--
Jérôme Marant <address@concealed>
<address@concealed>

http://marant.org




Archive powered by MHonArc 2.6.19+.

Top of Page