[Raw Msg Headers][Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

smtpserver problem and fix...




  I've noticed that domain names with all numeric components like 
1040.ship.net are not acceptable.  It appears to be a bug since comments 
in rfc821_domain() indicate that "1302.watstar.waterloo.edu" should be 
acceptable.

  The following diff does two things:

  - removes a bogus comment about acceptable domains

  - changes (!strict && !has_alpha) to (strict && !has_alpha) so that all 
numberic componets are acceptable when not operating "strict"



Tom

*** rfc821scn.c.orig	Tue Jul  4 17:50:27 1995
--- rfc821scn.c	Tue Jul  4 17:52:00 1995
***************
*** 427,436 ****
  
  	if (!s || !*s) return s; /* Pathological termination */
  
- 	/* The first test should be  is_821_alpha(), as per RFC821
- 	   we should accept only A-Z,a-Z in the begining, but
- 	   3COM spoiled that... Grr...				*/
- 
  	if (!is_821_alnum(*p)) {
  	  rfc821_error_ptr = p;
  	  rfc821_error = bad_name;
--- 427,432 ----
***************
*** 445,451 ****
  
  	*allnump &= !has_alpha;
  
! 	if (!is_821_alnum(c) || (!strict && !has_alpha)) {
  	  rfc821_error_ptr = p-1;
  	  rfc821_error = bad_name;
  	  return s; /* Bad <name>, don't advance */
--- 441,447 ----
  
  	*allnump &= !has_alpha;
  
! 	if (!is_821_alnum(c) || (strict && !has_alpha)) {
  	  rfc821_error_ptr = p-1;
  	  rfc821_error = bad_name;
  	  return s; /* Bad <name>, don't advance */