修正Qmail auth smtp中电子邮件地址任意的patch

gadfly
修正Qmail auth smtp中电子邮件地址任意的patch

qmail 的auth smtp有两个patch,
分别在:http://members.elysium.pl/brush/qmail-smtpd-auth/和www.nimh.org/hacks/qmail-smtpd.c。

不过都有个问题,就是auth 的用户可以和mail from的用户不一样,这样就
给了垃圾邮件发送者可乘之机,只要知道了某个用户的地址和口令,就可以
以自己的名义发信。同时认证用户的过程并不记录在邮件日志中,这给了系统管理员造成了很大不便,难以知道是哪个用户被黑掉了。

下面这个patch就是针对这种情况写的。

作用有两个:
1. 通过环境变量来控制是否记录认证过程。
2. 如果mail from和auth user name不一致,拒绝发送信件。


这个patch是基于www.nimh.org/hacks/qmail-smtpd.c,iceblood版本中实际上也是的这个。因此,用iceblood版本的可以直接使用。如果是用
http://members.elysium.pl/brush/qmail-smtpd-auth/版本的就需要自己
相应的修改了。

安装方法,如下,将下面这些内容存到qmail-smtpd.patch。放到qmail的源码目录。然后
1. patch < qmail-smtpd.patch
2. make qmail-smtpd
3. 替换qmail-smtpd
   或者复制到新的执行文件例如qmail-smtpd.auth,这样你的旧版本就能保留
  了,只需要在smtpd的启动脚本中,修改为对应的执行文件就可以。

如果你需要记录smtp认证的信息,在smtpd的启动脚本中定义环境变量,如下:
LOG_AUTH=1
export LOG_AUTH

缺省情况下,认证信息不记入log

[quote]
--- qmail-smtpd.c       2003-04-05 17:55:20.000000000 -0500
+++ qmail-smtpd.c.new   2003-04-05 17:50:58.000000000 -0500
@@ -304,7 +304,11 @@
   if (!stralloc_copys(&rcptto,"")) die_nomem();
   if (!stralloc_copys(&mailfrom,addr.s)) die_nomem();
   if (!stralloc_0(&mailfrom)) die_nomem();
+  if (smtp_auth_validfrom(mailfrom.s)) {
   out("250 ok/r/n");
+  } else {
+    out("must use username as From authenticated! (#5.7.1)/r/n");
+  }
}
void smtp_rcpt(arg) char *arg; {
   if (!seenmail) { err_wantmail(); return; }
@@ -527,6 +531,82 @@
static stralloc smtpauth = {0};
static char smtpauthlogin[65];
static char smtpauthpass[65];
+static int authd = 0;
+
+/* Author: gadfly@163.com.
+ * 1. Check consistent between auth user and 'From' user.
+ * 2. Read the LOG_AUTH enviroment variable to determine whether logging the auth info.
+ * LOG_AUTH=1, write the auth smtp info into syslog.
+ */
+#include <syslog.h>;
+
+#define SMTP_AUTH_SUCCESS     0
+#define SMTP_AUTH_FAILED      1
+
+int smtp_auth_validfrom(from) char * from;
+{
+  stralloc tmplogin={0},mydefaultdomain={0};
+  char authusername[256];
+  int k, userlen;
+
+  if (!authd) return 1;
+
+  authusername[255] = '/0';
+  userlen = k = str_len(smtpauthlogin);
+  k = byte_rchr(smtpauthlogin, k, '@');
+
+  if (k == userlen)
+  {
+    k = byte_rchr(smtpauthlogin, userlen, '%');
+
+    if (k == userlen)
+    {
+      if (control_readfile(&mydefaultdomain,"/var/qmail/control/me",1) != 1)
+      {
+        die_nomem();
+      }
+      if (!stralloc_copys(&tmplogin, smtpauthlogin) )
+      {
+        die_nomem();
+      }
+      if (!stralloc_cats(&tmplogin, "@") )
+      {
+        die_nomem();
+      }
+      if (!stralloc_cat(&tmplogin, &mydefaultdomain))
+      {
+        die_nomem();
+      }
+      strncpy(authusername, tmplogin.s, sizeof(authusername));
+      }
+      else
+      {
+        strcpy(authusername, smtpauthlogin);
+      authusername[k]='@';
+      }
+  }
+  else {
+    strcpy(authusername, smtpauthlogin);
+  }
+  return !strcasecmp(from, authusername);
+}
+
+
+void auth_smtplog(authlogin, authresult)
+char * authlogin;
+int authresult;
+{
+  char * x = env_get("LOG_AUTH");
+
+  if (!x || (*x != '1')) return;
+
+  if (authresult == SMTP_AUTH_SUCCESS) {
+    syslog(LOG_MAIL | LOG_INFO, "smtp auth: user name is %s, success!", authlogin);
+  } else {
+    syslog(LOG_MAIL | LOG_INFO, "smtp auth: user name is %s, failed!", authlogin);
+  }
+}
+
static int smtpauth_getl(void) {
   int i;
   if (!stralloc_copys(&smtpauth, "")) return -1;
@@ -611,11 +691,14 @@
   wait_pid(&st, pid);
   if (wait_exitcode(st) == 0) {
     out("235 go ahead/r/n");
+    authd = 1;
+    auth_smtplog(smtpauthlogin, SMTP_AUTH_SUCCESS);
     flush();
     relayclient="";
     return;
   }
   sleep(2);
+  auth_smtplog(smtpauthlogin, SMTP_AUTH_FAILED);
   out("535 auth failure/r/n"); flush(); _exit(0);
   /* done */
}
[/quote]

呵呵,正式用之前,各位多测试测试。

几种情况需要测测看:
1. 其它服务器发到本域,smtp过程是否正常,本域能否收到
2. from和auth user 不一样会出现什么情况, 一样的话,能否发信。
3. LOG_AUTH是否起作用。

呵呵,qmail的对齐用空格,实在不习惯,没办法,为保持一致,我也用了。     

[b]更新版本[/b]
应一些朋友的要求,加了本地发信人发给本地,也需要auth的功能。

另外而且还加了一些注释,版本号。方法和前面一样。

代码如下:
[code]
--- qmail-smtpd.c        2003-06-02 18:34:51.000000000 -0400
+++ qmail-smtpd.c.new        2003-06-02 18:42:31.000000000 -0400
@@ -268,6 +268,7 @@
   int r;
   r = rcpthosts(addr.s,str_len(addr.s));
   if (r == -1) die_control();
+  if (!localauthd()) return 0;
   return r;
}

@@ -304,7 +305,11 @@
   if (!stralloc_copys(&rcptto,"")) die_nomem();
   if (!stralloc_copys(&mailfrom,addr.s)) die_nomem();
   if (!stralloc_0(&mailfrom)) die_nomem();
-  out("250 ok/r/n");
+  if (smtp_auth_validfrom(mailfrom.s)) {
+    out("250 ok/r/n");
+  } else {
+    out("must use username as From authenticated! (#5.7.1)/r/n");
+  }
}
void smtp_rcpt(arg) char *arg; {
   if (!seenmail) { err_wantmail(); return; }
@@ -527,6 +532,89 @@
static stralloc smtpauth = {0};
static char smtpauthlogin[65];
static char smtpauthpass[65];
+static int authd = 0;
+
+/* Author: gadfly@163.com. Version: 1.2.
+ * 1. Check consistent between auth user and 'From' user.
+ * 2. Read the LOG_AUTH enviroment variable to determine whether logging the auth info.
+ * LOG_AUTH=1, write the auth smtp info into syslog.
+ * 3. If mail from local, rcpt to local domain, user must auth before send data.
+ */
+#include <syslog.h>;
+
+#define SMTP_AUTH_SUCCESS     0
+#define SMTP_AUTH_FAILED      1
+
+int localauthd() {
+
+  if (rcpthosts(mailfrom.s, strlen(mailfrom.s)) && !authd) return 0;
+  return 1;
+}
+
+int smtp_auth_validfrom(from) char * from;
+{
+  stralloc tmplogin={0},mydefaultdomain={0};
+  char authusername[256];
+  int k, userlen;
+
+  if (!authd) return 1;
+
+  authusername[255] = '/0';
+  userlen = k = str_len(smtpauthlogin);
+  k = byte_rchr(smtpauthlogin, k, '@');
+
+  if (k == userlen)
+  {
+    k = byte_rchr(smtpauthlogin, userlen, '%');
+
+    if (k == userlen)
+    {
+      if (control_readfile(&mydefaultdomain,"/var/qmail/control/me",1) != 1)
+      {
+        die_nomem();
+      }
+      if (!stralloc_copys(&tmplogin, smtpauthlogin) )
+      {
+        die_nomem();
+      }
+      if (!stralloc_cats(&tmplogin, "@") )
+      {
+        die_nomem();
+      }
+      if (!stralloc_cat(&tmplogin, &mydefaultdomain))
+      {
+        die_nomem();
+      }
+      strncpy(authusername, tmplogin.s, sizeof(authusername));
+      }
+      else
+      {
+        strcpy(authusername, smtpauthlogin);
+      authusername[k]='@';
+      }
+  }
+  else {
+    strcpy(authusername, smtpauthlogin);
+  }
+  return !strcasecmp(from, authusername);
+}
+
+
+void auth_smtplog(authlogin, authresult)
+char * authlogin;
+int authresult;
+{
+  char * x = env_get("LOG_AUTH");
+
+  if (!x || (*x != '1')) return;
+
+  if (authresult == SMTP_AUTH_SUCCESS) {
+    syslog(LOG_MAIL | LOG_INFO, "smtp auth: user name is %s, success!", authlogin);
+  } else {
+    syslog(LOG_MAIL | LOG_INFO, "smtp auth: user name is %s, failed!", authlogin);
+  }
+}
+
static int smtpauth_getl(void) {
   int i;
   if (!stralloc_copys(&smtpauth, "")) return -1;
@@ -611,11 +699,14 @@
   wait_pid(&st, pid);
   if (wait_exitcode(st) == 0) {
     out("235 go ahead/r/n");
+    authd = 1;
+    auth_smtplog(smtpauthlogin, SMTP_AUTH_SUCCESS);
     flush();
     relayclient="";
     return;
   }
   sleep(2);
+  auth_smtplog(smtpauthlogin, SMTP_AUTH_FAILED);
   out("535 auth failure/r/n"); flush(); _exit(0);
   /* done */
}
[/code]

另外,感谢erehw提供的空间可以下载这个patch。
http://gadfly.shanji.com/qmail-smtpd/

[b]
由于网页上的代码贴上去,格式有问题。
我就不再更新了。
请各位到以上的url下载。
或给我发信索要
[/b]

Fun-FreeBSD
修正Qmail auth smtp中电子邮件地址任意的patch

好啊,可惜我没在公司,周一到公司试一下~

peng
修正Qmail auth smtp中电子邮件地址任意的patch

gadfly真强啊!
看来做研发的就是厉害啊。
好文章。

gadfly
修正Qmail auth smtp中电子邮件地址任意的patch

晕,这上面的patch中,每行前面的空格都不显示出来,复制下来肯定没法用.

本站的ftp服务也关了.

如果手工复制下来,需要在每行没有@或+开头的代码行前面加个空格,
就可以使用了.

如果觉得麻烦,可以给我发信,我把patch发给你们. gadfly@163.com

最好能有个上载的地方 :-(

gadfly
修正Qmail auth smtp中电子邮件地址任意的patch

peng老大过奖了,都在技术的苦海中苦苦挣扎. 兄弟们得携手共进才是.

说不定还有不少问题呢

gadfly
修正Qmail auth smtp中电子邮件地址任意的patch

由于,有网友给我发信提了一些问题,我想还是到这里说比较方便。

其实下面的问题,在我上面都说明了。没办法,我还是重点说一下

FAQ:

1. patch是基于哪一个auth的?
auth smtp有两个patch,
分别在:http://members.elysium.pl/brush/qmail-smtpd-auth/
和www.nimh.org/hacks/qmail-smtpd.c。
基于的是后者。
如果你们auth smtp用的是前者,我提供的patch就不能直接使用。
等我有时间,在写一个基于前者的patch吧。

2. patch执行后的结果是什么?
晕,刚发现,www.nimh.org/hacks/qmail-smtpd.c又更新了,做的糟糕,没有版本信息,我还不知道我针对的哪一个版本。下次我也要加上版本信息了。
现在只好说: iceblood版本: 正常执行成功的话,没有任何输出。
          最新版本(nimh.org主页上的):输出入下:
[quote]
Hunk #1 succeeded at 252 (offset -52 lines).
Hunk #2 succeeded at 498 (offset -33 lines).
Hunk #3 succeeded at 633 (offset -58 lines).
[/quote]
但是还好,编译出来的可以正常使用。

3.编译出来的大小没有区别?
这个和系统环境相关,不是绝对的。只要前两个没问题,你就看测试结果吧。

4.网页上的patch不好用?
是的,因为格式的原因,手工复制下来,需要在每行没有@或+开头的代码行前面加个空格, 就可以使用了.

如果觉得麻烦,可以给我发信,我把patch发给你们. gadfly@163.com

最好能有个上载的地方

netloafer
修正Qmail auth smtp中电子邮件地址任意的patch

gadfly真是强呀~~
有关资料可以上传到linux版的FTP服务器上~~~~

gadfly
修正Qmail auth smtp中电子邮件地址任意的patch

[quote][i]原帖由 "netloafer" 发表:[/i]


gadfly真是强呀~~
有关资料可以上传到linux版的FTP服务器上~~~~


[/quote]

呵呵,过奖了。

ftpserver我昨天试过了,还是没有开。

听说因为受攻击,还影响了电信的其他业务,现在已经关闭了。只有另寻他途。

夜未眠
修正Qmail auth smtp中电子邮件地址任意的patch

针对http://members.elysium.pl/brush/qmail-smtpd-auth/认证补丁再加上楼主所说的功能.

[ 非patch ]

首先你要打补丁,然后照下面的方法修改qmail-smtpd.c源代码.

1.增加全局变量:
        stralloc authusername={0};   //全局变量用于记录认证的email地址

2.在认证成功后将mailfrom存入上面的authusername中,也就是在函数
int authenticate(void)最后return 0;语句前插入下面的语句
         /*将用户名存入全局变量 authusername*/
         if(!stralloc_copy(&authusername,&user)) return 1;

3.在void smtp_rcpt(arg)函数的if (relayclient) {语句后插入下面的语句.
             if(case_diffs(authusername.s,mailfrom.s)) {
                    err_noauth(); //认证用户名与mailfrom中的地址不符
                    /*在这儿加入log代码*/
                    return; //出错返回
             }

===========================================================
一个问题:也不知道我想的对不对,在此提出与各位讨论.

    但是我觉得你的patch和我上面的方法好象都有一个问题.就是如果远程连接到服务器25号端口,不通过认证。一样可以向control/rcpthosts中列出的地址发邮件(就象是别的MTA与你对话一样,别人的MTA不可能知道你的用户名和密码)因为不通过认证那么relayclient将会是null,那么就会进行rcpthosts的比较,如果rcpt 指定的email地址在rcpthosts中列出,qmail会接收并转发该邮件,如果一个快速请求对一个慢速服务器,那是什么情况?
     有人想把用户名与mailfrom地址比较的操作放在if(relayclient)这前,前面我说了,别人的MTA不会知道你的用户名及密码。如果你这样作了,你将收不到邮件。
    我想就一台服务器再打补丁也是白打。可以用两台服务器比较安全,分别打不同的补丁<也就是在前面提到的if(relayclient)处>;,一台专收,一台专发。两台之间进行通信。这样可能才能比较安全.

    不知道各位大虾有什么看法。不仿在此讨论讨论..

tutux
修正Qmail auth smtp中电子邮件地址任意的patch

[quote][i]原帖由 "夜未眠"]  发表:[/i]


首先你要打补丁,然后照下面的方法修改qmail-smtpd.c源代码.

1.增加全局变量:
        stralloc authusername={0};   //全局变量用于记录认证的email地址

2.在认证成功后将mailfrom存入上面的authusern..........[/quote]

[quote]就是如果远程连接到服务器25号端口,不通过认证。一样可以向control/rcpthosts中列出的地址发邮件(就象是别的MTA与你对话一样,别人的MTA不可能知道你的用户名和密码)因为不通过认证那么relayclient将会是null,那么就会进行rcpthosts的比较,如果rcpt 指定的email地址在rcpthosts中列出,qmail会接收并转发该邮件,如果一个快速请求对一个慢速服务器,那是什么情况?
[/quote]

这个情况,是暂时没办法的啦。

iceblood
修正Qmail auth smtp中电子邮件地址任意的patch

谢谢,希望加精。
能否mail给我一份?
我尽快把qmail-smtpd.c替换掉。

gadfly
修正Qmail auth smtp中电子邮件地址任意的patch

to 夜未眠:

大概明白你的意思.
1.smtp的这种机制下,安全就是相对的,我知道你服务器的域和用户,我就可以向你发信。
2.其实你说的分开,也就是将mail relay(本域用户向外发信)和mail receive(用于MTA)之间,我记得和peng以前讨论过,确实可以这么做.而且不同的patch确实能使效率提高,同时又起到了分流的作用.不过大部分的情况下,效率的问题并不大, 这两者还是合一的.不过我想用不同的patch,效率上的提高还是有限的。即使有效率问题,完全可以通过多服务器负载均衡来避免.
3.你说白打,我的理解你是针对发往本域的,我同意。不过这些patch基本上都是针对relay的,从这个角度来说,patch是很有作用的.

to iceblood:
看不到你的mail address。不知如何给你

iceblood
修正Qmail auth smtp中电子邮件地址任意的patch

晕到~一时忽疏~居然忘记留下地址,现在补上~
iceblood@163.com

夜未眠
修正Qmail auth smtp中电子邮件地址任意的patch

to gadfly
我都不是很清楚自已在说什么,谁叫我表达能力太差呢。大哥居然能够理解我的意图,佩服佩服。

haohaoo
修正Qmail auth smtp中电子邮件地址任意的patch

[quote]这个patch是基于www.nimh.org/hacks/qmail-smtpd.c,iceblood版本中实际上也是的这个。因此,用iceblood版本的可以直接使用。[/quote]
那要怎么用?你还没有说呢?下载qmail-smtpd.c后要怎么办?我用iceblood版本已经安装好了,要怎么打?

gadfly
修正Qmail auth smtp中电子邮件地址任意的patch

不仔细看?
[quote]
安装方法,如下,将下面这些内容存到qmail-smtpd.patch。放到qmail的源码目录。然后
1. patch < qmail-smtpd.patch
2. make qmail-smtpd
3. 替换qmail-smtpd
[/quote]

haohaoo
修正Qmail auth smtp中电子邮件地址任意的patch

[quote][i]原帖由 "gadfly"][/quote 发表:[/i]

有看啊,不过我说iceblood版本已经安装好了的啊?
是不是把包解开,然后把qmail-smtpd.c改名,然后编译好后,把qmail-smtpd拿到已安装的机器上去替换就行了?

gadfly
修正Qmail auth smtp中电子邮件地址任意的patch

??
不用改名,解包后按我说的步骤作就好了

haohaoo
修正Qmail auth smtp中电子邮件地址任意的patch

我替换掉qmail-smtpd后还是出问题啊,smtp不用验证就可以发信啊

ryanxyz
修正Qmail auth smtp中电子邮件地址任意的patch

我一開始就用了gadfly兄的patch
完全沒有問題
一次搞定 如果是從網頁上複製下來的
需要注意空白的問題
上一篇:telnet 110 连接失败 下一篇:ChinaUnix.net