Re: [vchkpw] Problem with add_alias

2007-09-06 Thread Joshua Megerman
On Thursday 06 September 2007 01:18:32 am Rick Widmer wrote:
 Alessio Cecchi wrote:
  I'm using vpopmail 5.4.20 with vpopmaild enable.
 
  When i add an alias from a telnet session:
 
  [EMAIL PROTECTED]:~$ telnet mail.pippo.com 89
  Trying 83.49.65.71...
  Connected to mail.pippo.com.
  Escape character is '^]'.
  +OK
  login [EMAIL PROTECTED] X
  +OK+
  vpopmail_dir /home/vpopmail
  domain_dir /home/vpopmail/domains/pippo.com
  [...]
  no_maildrop 0
  system_admin_privileges 0
  .
  add_alias [EMAIL PROTECTED] [EMAIL PROTECTED]
  +OK
  quit
  +OK
  Connection closed by foreign host.
 
  After i'm unable to send e-mail to the alias:
 
  Hi. This is the qmail-send program at s1.pippo.com.
  I'm afraid I wasn't able to deliver your message to the following
  addresses. This is a permanent error; I've given up. Sorry it didn't work
  out.
 
  [EMAIL PROTECTED]:
  Sorry, I couldn't find any host named server.com?. (#5.1.2)
 
  --- Below this line is a copy of the message.
 
  [...]
 
  @400046dea9ab22704ab4 delivery 2089: failure:
  Sorry,_I_couldn't_find_any_host_named_server.com?._(#5.1.2)/
  @400046dea9ab22738ea4 status: local 0/10 remote 0/20
 
 
  Please note the problem of the domain name:
 
  server.com?.

 That is where you told it to send mail received by [EMAIL PROTECTED]:

add_alias [EMAIL PROTECTED] [EMAIL PROTECTED]

I think he's obscuring the real domain name, but the key here is the '?' 
after server.com and before the final '.' - it's how qmail indicates a 
funky character IIRC.

  Why this?

 server.com doesn't have any MX records, so qmail can't find a host to
 send the message to.

But it does have an A record, so if he really was trying to send 
to server.com, qmail should have responded with a can't establish an SMTP 
connection message instead.

  Could be the problem relate to the crlf of the telnete session?

 No.

I agree that it's unlikely, but maybe something else is happening.  But I 
agree that the original poster needs to provide more info, preferably without 
hiding anything (other than passwords), so we can diagnose it better.  Also, 
the output of valias showing the alias, and perhaps the actual alias DB dump 
showing it would be helpful too...

Josh
-- 
Joshua Megerman
SJGames MIB #5273 - OGRE AI Testing Division
You can't win; You can't break even; You can't even quit the game.
  - Layman's translation of the Laws of Thermodynamics
[EMAIL PROTECTED]


Re: [vchkpw] Problem with add_alias

2007-09-06 Thread Alessio Cecchi
On Thursday 06 September 2007 07:18:32 Rick Widmer wrote:
  @400046dea9ab22704ab4 delivery 2089: failure:
  Sorry,_I_couldn't_find_any_host_named_server.com?._(#5.1.2)/
  @400046dea9ab22738ea4 status: local 0/10 remote 0/20
 
 
  Please note the problem of the domain name:
 
  server.com?.

 That is where you told it to send mail received by [EMAIL PROTECTED]:

    add_alias [EMAIL PROTECTED] [EMAIL PROTECTED]

  Why this?

 server.com doesn't have any MX records, so qmail can't find a host to
 send the message to.

  Could be the problem relate to the crlf of the telnete session?

 No.

Sorry Rick but the problem is crlf, the domain are only for example (on my 
server i have used real domain name like gmail.com), but the problem is 
the .qmail generated by the add_alias, infact if i run dos2unix .qmail-prova 
the alias works.

Also if i open the same .qmail vith vim i see [DOS] for encoding.

Thanks
-- 
Alessio Cecchi is:
@ ILS - http://www.linux.it/~alessice/
Assistenza Sistemi GNU/Linux - http://www.cecchi.biz/
@ PLUG - Presidente, http://www.prato.linux.it


Re: [vchkpw] Problem with add_alias

2007-09-06 Thread Joshua Megerman
  Could be the problem relate to the crlf of the telnete session?

 No.

 Sorry Rick but the problem is crlf, the domain are only for example (on my
 server i have used real domain name like gmail.com), but the problem is
 the .qmail generated by the add_alias, infact if i run dos2unix
 .qmail-prova
 the alias works.

 Also if i open the same .qmail vith vim i see [DOS] for encoding.

Actually, looking at the code, he's right - the line in question is in
vpopmaild.c, line 1002:

if ((alias_line=strtok(NULL, \n))==NULL) {

The strtok tokenizes after the '\n' (LF), leaving the '\r' (CR) as the
last character of the alias_line, which would have the exact effect he's
seeing.  Looks like the same bug exists on line 1051 as well (identical
code block in remove_alias).  Every other instance of strtok() uses one of
two defines for the token list - TOKENS or PASS_TOKENS, these are the only
two strtok() calls using an explicit token list.  Patch is attached.

Josh
Joshua Megerman
SJGames MIB #5273 - OGRE AI Testing Division
You can't win; You can't break even; You can't even quit the game.
  - Layman's translation of the Laws of Thermodynamics
[EMAIL PROTECTED]--- ../vpopmail-5.4.20/vpopmaild.c	2007-08-22 02:17:45.0 -0400
+++ vpopmaild.c	2007-09-06 09:47:04.0 -0400
@@ -999,7 +999,7 @@
 return(-1);
   }
 
-  if ((alias_line=strtok(NULL, \n))==NULL) {
+  if ((alias_line=strtok(NULL,TOKENS))==NULL) {
 show_error( ERR_ALIAS_LINE_REQD, 906 );
 return(-1);
   }
@@ -1048,7 +1048,7 @@
 return(-1);
   }
 
-  if ((alias_line=strtok(NULL, \n))==NULL) {
+  if ((alias_line=strtok(NULL,TOKENS))==NULL) {
 show_error( ERR_ALIAS_LINE_REQD, 1006 );
 return(-1);
   }

Re: [vchkpw] Problem with add_alias

2007-09-06 Thread Alessio Cecchi
On Thursday 06 September 2007 15:48:56 Joshua Megerman wrote:
  Also if i open the same .qmail vith vim i see [DOS] for encoding.

 Actually, looking at the code, he's right - the line in question is in
 vpopmaild.c, line 1002:

     if ((alias_line=strtok(NULL, \n))==NULL) {

 The strtok tokenizes after the '\n' (LF), leaving the '\r' (CR) as the
 last character of the alias_line, which would have the exact effect he's
 seeing.  Looks like the same bug exists on line 1051 as well (identical
 code block in remove_alias).  Every other instance of strtok() uses one of
 two defines for the token list - TOKENS or PASS_TOKENS, these are the only
 two strtok() calls using an explicit token list.  Patch is attached.

Thank you Joshua,

with this patch now add_alias from telnet works!

Bye
-- 
Alessio Cecchi is:
@ ILS - http://www.linux.it/~alessice/
Assistenza Sistemi GNU/Linux - http://www.cecchi.biz/
@ PLUG - Presidente, http://www.prato.linux.it


Re: [vchkpw] Problem with add_alias

2007-09-06 Thread Rick Widmer
Thank you both!  The patch has been added to CVS, and will appear in the 
next release.



Rick


Re: [vchkpw] Re: [vpopmail-devel] vpopmail 5.4.21 released

2007-09-06 Thread Rick Widmer



Joshua Megerman wrote:

Alessio Cecchi found a bug in vpopmaild add_alias command, and Joshua
Megerman provided a patch.  If you are using vpopmaild you probably want
to upgrade...


You spelled my name correctly above...


Changelog:

5.4.21 - Released 06-Sep-2007
Joshua Mergeman, Alessio Cecchi
- fix bug in vpopmaild add_alias command


But not in the Changelog... :(  Kindly fix it and repost the release if
you could :)


Done...


[vchkpw] Decided to make the plunge today on late version of vpopmail

2007-09-06 Thread Steve Cole
On two (low volume) machines with vpopmail interfaced with mysql as the data 
store, it seemed to work successfully.  I updated the database schema with no 
issues.  This is with v5.4.21

However, when I do a vdeluser on either machine, the program segfaults.  On 
both systems.

I tried various things like making sure my LDFLAGS= and CFLAGS=-O only to 
be sure it wasn't a compiler issue, without luck.

So, to be clear, this is on two Debian Etch machines with GCC 4.1.1-15 and 
MySQL 5.0.32-7etch1 installed.  

Going back to 5.4.17 seemed to be no issue at all and of course, it works as 
expected.

-- 
--
Cheers,
Steve


[vchkpw] Re: [vpopmail-devel] vpopmail 5.4.21 released

2007-09-06 Thread Joshua Megerman
 Alessio Cecchi found a bug in vpopmaild add_alias command, and Joshua
 Megerman provided a patch.  If you are using vpopmaild you probably want
 to upgrade...

You spelled my name correctly above...

 Changelog:

 5.4.21 - Released 06-Sep-2007
   Joshua Mergeman, Alessio Cecchi
   - fix bug in vpopmaild add_alias command

But not in the Changelog... :(  Kindly fix it and repost the release if
you could :)

Thanks,
Josh

Joshua Megerman
SJGames MIB #5273 - OGRE AI Testing Division
You can't win; You can't break even; You can't even quit the game.
  - Layman's translation of the Laws of Thermodynamics
[EMAIL PROTECTED]



[vchkpw] vpopmail 5.4.21 released

2007-09-06 Thread Rick Widmer

http://vpopmail.sf.net

5.4.21 - released 6 Sept 2007

Release Notes:

Alessio Cecchi found a bug in vpopmaild add_alias command, and Joshua 
Megerman provided a patch.  If you are using vpopmaild you probably want 
to upgrade...



Changelog:

5.4.21 - Released 06-Sep-2007
Joshua Mergeman, Alessio Cecchi
- fix bug in vpopmaild add_alias command



NOTE:  This release contains only the patch Joshua just posted to the 
vpopmail list...


Re: [vchkpw] Decided to make the plunge today on late version of vpopmail

2007-09-06 Thread Joshua Megerman

 On two (low volume) machines with vpopmail interfaced with mysql as the
 data
 store, it seemed to work successfully.  I updated the database schema with
 no
 issues.  This is with v5.4.21

Did you follow the UPGRADE document in 5.4.21 exactly and rename the
'domain' columns to 'pw_domain'?  If so, then your DB is now wrong. 
Apparently the fix I posted about the UPGRADE document didn't make it into
5.4.21...

The vpopmail table, and ONLY the vpopmail table, has a column named
'pw_domain'.  Every other table should have a 'domain' column, as the
earlier UPGRADE document listed (which incorrectly listed vpopmail as
having a 'domain' column).

The other suggestion I can make is try stracing the new version and see
where it fails...

Josh

Joshua Megerman
SJGames MIB #5273 - OGRE AI Testing Division
You can't win; You can't break even; You can't even quit the game.
  - Layman's translation of the Laws of Thermodynamics
[EMAIL PROTECTED]



Re: [vchkpw] Decided to make the plunge today on late version of vpopmail

2007-09-06 Thread Steve Cole
On Thursday 06 September 2007, Joshua Megerman wrote:

 The vpopmail table, and ONLY the vpopmail table, has a column named
 'pw_domain'.  Every other table should have a 'domain' column, as the
 earlier UPGRADE document listed (which incorrectly listed vpopmail as
 having a 'domain' column).

No, thankfully.  I was the one who reported the inconsistency in the upgrade 
document when *.20 rolled.  I should have mentioned that this same thing 
happened to one of my test boxes when I tried *.20 but it didn't on another 
(which it turns out is CDB because it handles two personal domains).  So *.20 
likely has the problem as well, I didn't delve that deeply the first time.

I straced the command and it opens a MySQL connection, does a MySQL query and 
then segfaults immediately.

As I said, the behaviour doesn't occur in the CDB binary.

-- 
--
Cheers,
Steve


Re: [vchkpw] Decided to make the plunge today on late version of vpopmail

2007-09-06 Thread Quey

Rick Widmer wrote:



Steve Cole wrote:
On two (low volume) machines with vpopmail interfaced with mysql as 
the data store, it seemed to work successfully.  I updated the 
database schema with no issues.  This is with v5.4.21


However, when I do a vdeluser on either machine, the program 
segfaults.  On both systems.


I tried it here with --enable-auth-module=mysql but don't get the 
segfault.  So can you send me:


   The vpopmail ./configure options you are using

   The actual user/domain you are trying to delete

   The strace output

You can send it private if you wish...


Going back to 5.4.17 seemed to be no issue at all and of course, it 
works as expected.


Nothing within vdeluser changed between those versions, but there were 
changes in vpopmail.c, but nothing that stands out.



Rick Widmer wrote:



Steve Cole wrote:
On two (low volume) machines with vpopmail interfaced with mysql as 
the data store, it seemed to work successfully.  I updated the 
database schema with no issues.  This is with v5.4.21


However, when I do a vdeluser on either machine, the program 
segfaults.  On both systems.


I tried it here with --enable-auth-module=mysql but don't get the 
segfault.  So can you send me:


   The vpopmail ./configure options you are using

   The actual user/domain you are trying to delete

   The strace output

You can send it private if you wish...


Going back to 5.4.17 seemed to be no issue at all and of course, it 
works as expected.


Nothing within vdeluser changed between those versions, but there were 
changes in vpopmail.c, but nothing that stands out.



Rick on a kind of related note, did you see one of my posts in the thread:
adduser processing times CDB from 27th of the 7th ?



Re: [vchkpw] Decided to make the plunge today on late version of vpopmail

2007-09-06 Thread Rick Widmer



Steve Cole wrote:
On two (low volume) machines with vpopmail interfaced with mysql as the data 
store, it seemed to work successfully.  I updated the database schema with no 
issues.  This is with v5.4.21


However, when I do a vdeluser on either machine, the program segfaults.  On 
both systems.


I tried it here with --enable-auth-module=mysql but don't get the 
segfault.  So can you send me:


   The vpopmail ./configure options you are using

   The actual user/domain you are trying to delete

   The strace output

You can send it private if you wish...


Going back to 5.4.17 seemed to be no issue at all and of course, it works as 
expected.


Nothing within vdeluser changed between those versions, but there were 
changes in vpopmail.c, but nothing that stands out.