aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog3
-rw-r--r--src/bi_sa.m4218
-rw-r--r--src/dnsbase.c4
3 files changed, 182 insertions, 43 deletions
diff --git a/ChangeLog b/ChangeLog
index c50cbd64..053fa59c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,7 @@
2007-10-21 Sergey Poznyakoff <gray@gnu.org.ua>
+ * src/dnsbase.c (cname_loop_body): Return TXT only if explicitly
+ required. Bug reported by Jan Rafaj.
+
* src/engine.c: Fix timeout calculations
(struct timeout_ctl): New data type
diff --git a/src/bi_sa.m4 b/src/bi_sa.m4
index 0ed7472b..8472bbe9 100644
--- a/src/bi_sa.m4
+++ b/src/bi_sa.m4
@@ -28,25 +28,148 @@ MF_VAR(sa_keywords, STRING);
MF_VAR(clamav_virus_name, STRING);
+
+struct mf_stream {
+ eval_environ_t env;
+ mu_stream_t str;
+ time_t start;
+ time_t timeout;
+};
+
+typedef struct mf_stream *mf_stream_t;
+
+/* FIXME: duplicated in engine.c */
+#define UPDATE_TTW(t,start) do { \
+ time_t delta = time(NULL) - start; \
+ if (t > delta) \
+ t -= delta; \
+ else \
+ t = 0; \
+} while (0)
+
+extern int smtp_stream_wait(mu_stream_t stream, int flags, time_t *timeout);
+static void spamd_destroy(mf_stream_t *pstream);
+
+mf_stream_t
+mf_stream_new (eval_environ_t env, mu_stream_t stream, time_t start,
+ time_t timeout)
+{
+ mf_stream_t pmfs = xmalloc (sizeof pmfs[0]);
+ pmfs->env = env;
+ pmfs->str = stream;
+ pmfs->start = start;
+ pmfs->timeout = timeout;
+ return pmfs;
+}
+
+int
+mf_stream_sequential_read(mf_stream_t stream, char *buf, size_t size)
+{
+ int rc;
+ eval_environ_t env = stream->env;
+
+ if (stream->timeout == 0) {
+ spamd_destroy(&stream);
+ MF_THROW(mf_temp_failure,
+ "sa_sequential_read: %s",
+ mu_strerror(ETIMEDOUT));
+ }
+
+ rc = smtp_stream_wait(stream->str, MU_STREAM_READY_RD,
+ &stream->timeout);
+ if (rc == 0) {
+ UPDATE_TTW(stream->timeout, stream->start);
+ rc = mu_stream_sequential_read(stream->str, buf, size, NULL);
+ } else {
+ spamd_destroy(&stream);
+ MF_THROW(mf_temp_failure,
+ "sa_sequential_read: %s",
+ mu_strerror(ETIMEDOUT));
+ }
+
+ return rc;
+}
+
+int
+mf_stream_sequential_readline(mf_stream_t stream, char *buf, size_t size,
+ size_t *nbytes)
+{
+ int rc;
+ eval_environ_t env = stream->env;
+
+ if (stream->timeout == 0) {
+ spamd_destroy(&stream);
+ MF_THROW(mf_temp_failure,
+ "sa_sequential_readline: %s",
+ mu_strerror(ETIMEDOUT));
+ }
+ rc = smtp_stream_wait(stream->str, MU_STREAM_READY_RD,
+ &stream->timeout);
+ if (rc == 0) {
+ UPDATE_TTW(stream->timeout, stream->start);
+ rc = mu_stream_sequential_readline(stream->str, buf, size,
+ nbytes);
+ } else {
+ spamd_destroy(&stream);
+ MF_THROW(mf_temp_failure,
+ "sa_sequential_readline: %s",
+ mu_strerror(ETIMEDOUT));
+ }
+ return rc;
+}
+
+int
+mf_stream_sequential_write(mf_stream_t stream, char *buf, size_t size)
+{
+ int rc;
+ eval_environ_t env = stream->env;
+
+ if (stream->timeout == 0) {
+ spamd_destroy(&stream);
+ MF_THROW(mf_temp_failure,
+ "sa_sequential_write: %s",
+ mu_strerror(ETIMEDOUT));
+ }
+
+ rc = smtp_stream_wait(stream->str, MU_STREAM_READY_WR,
+ &stream->timeout);
+ if (rc == 0) {
+ UPDATE_TTW(stream->timeout, stream->start);
+ rc = mu_stream_sequential_write(stream->str, buf, size);
+ } else {
+ spamd_destroy(&stream);
+ MF_THROW(mf_temp_failure,
+ "sa_sequential_write: %s",
+ mu_strerror(ETIMEDOUT));
+ }
+ return rc;
+}
+
+
+
static int
-spamd_connect_tcp(eval_environ_t env, mu_stream_t *stream,
- char *host, int port)
+spamd_connect_tcp(eval_environ_t env, mf_stream_t *pmfs,
+ char *host, int port, time_t timeout)
{
- int rc = mu_tcp_stream_create(stream, host, port, 0);
+ mu_stream_t stream;
+ int rc = mu_tcp_stream_create(&stream, host, port, 0);
MF_ASSERT(rc == 0, mf_failure,
"mu_tcp_stream_create: %s",
mu_strerror(rc));
- rc = mu_stream_open(*stream);
+ rc = mu_stream_open(stream);
MF_ASSERT(rc == 0, mf_failure,
"mu_stream_open: %s",
mu_strerror(rc));
+ *pmfs = mf_stream_new(env, stream, time(NULL), timeout);
return rc;
}
static int
-spamd_connect_socket(eval_environ_t env, mu_stream_t *stream, char *path)
+spamd_connect_socket(eval_environ_t env, mf_stream_t *pmfs, char *path,
+ time_t timeout)
{
int fd, rc;
FILE *fp;
struct sockaddr_un addr;
+ mu_stream_t stream;
fd = socket(PF_UNIX, SOCK_STREAM, 0);
@@ -64,5 +187,5 @@ spamd_connect_socket(eval_environ_t env, mu_stream_t *stream, char *path)
fp = fdopen(fd, "w+");
- rc = mu_stdio_stream_create(stream, fp, MU_STREAM_RDWR);
+ rc = mu_stdio_stream_create(&stream, fp, MU_STREAM_RDWR);
if (rc) {
fclose(fp);
@@ -72,7 +195,7 @@ spamd_connect_socket(eval_environ_t env, mu_stream_t *stream, char *path)
}
- rc = mu_stream_open(*stream);
+ rc = mu_stream_open(stream);
if (rc) {
- mu_stream_destroy (stream, mu_stream_get_owner (*stream));
+ mu_stream_destroy(&stream, mu_stream_get_owner(stream));
fclose(fp);
MF_THROW(mf_failure,
@@ -80,15 +203,18 @@ spamd_connect_socket(eval_environ_t env, mu_stream_t *stream, char *path)
mu_strerror(rc));
}
+
+ *pmfs = mf_stream_new(env, stream, time(NULL), timeout);
+
return rc;
}
static void
-spamd_shutdown(mu_stream_t stream, int isfile, int flag)
+spamd_shutdown(mf_stream_t stream, int isfile, int flag)
{
int fd;
-
mu_transport_t trans;
- mu_stream_flush(stream);
- mu_stream_get_transport(stream, &trans);
+
+ mu_stream_flush(stream->str);
+ mu_stream_get_transport(stream->str, &trans);
if (isfile)
fd = fileno((FILE*)trans);
@@ -99,12 +225,15 @@ spamd_shutdown(mu_stream_t stream, int isfile, int flag)
static void
-spamd_destroy(mu_stream_t *stream)
+spamd_destroy(mf_stream_t *pstream)
{
- mu_stream_close(*stream);
- mu_stream_destroy(stream, mu_stream_get_owner(*stream));
+ mf_stream_t stream = *pstream;
+ mu_stream_close(stream->str);
+ mu_stream_destroy(&stream->str, mu_stream_get_owner(stream->str));
+ free(stream);
+ *pstream = NULL;
}
static void
-spamd_send_command(mu_stream_t stream, const char *fmt, ...)
+spamd_send_command(mf_stream_t stream, const char *fmt, ...)
{
char buf[512];
@@ -115,10 +244,10 @@ spamd_send_command(mu_stream_t stream, const char *fmt, ...)
n = vsnprintf (buf, sizeof buf, fmt, ap);
va_end (ap);
- mu_stream_sequential_write(stream, buf, n);
- mu_stream_sequential_write(stream, "\r\n", 2);
+ mf_stream_sequential_write(stream, buf, n);
+ mf_stream_sequential_write(stream, "\r\n", 2);
}
static void
-spamd_send_stream(mu_stream_t stream, mu_stream_t instr)
+spamd_send_stream(mf_stream_t stream, mu_stream_t instr)
{
size_t size;
@@ -130,13 +259,13 @@ spamd_send_stream(mu_stream_t stream, mu_stream_t instr)
&& size > 0) {
debug3(80,"<< %*.*s", size, size, buf);
- mu_stream_sequential_write (stream, buf, size);
+ mf_stream_sequential_write(stream, buf, size);
}
}
static int
-spamd_read_line(mu_stream_t stream, char *buffer, size_t size, size_t *pn)
+spamd_read_line(mf_stream_t stream, char *buffer, size_t size, size_t *pn)
{
size_t n = 0;
- int rc = mu_stream_sequential_readline(stream, buffer, size, &n);
+ int rc = mf_stream_sequential_readline(stream, buffer, size, &n);
if (rc == 0) {
if (pn)
@@ -188,9 +317,9 @@ decode_float(long *vn, char *str, int digits)
static int
-decode_boolean (char *str)
+decode_boolean(char *str)
{
- if (strcasecmp (str, "true") == 0)
+ if (strcasecmp(str, "true") == 0)
return 1;
- else if (strcasecmp (str, "false") == 0)
+ else if (strcasecmp(str, "false") == 0)
return 0;
/*else?*/
@@ -223,10 +352,11 @@ sigpipe_handler (int sig)
}
-mu_stream_t
-open_connection(eval_environ_t env, char *urlstr, int *isfile, char **phost)
+mf_stream_t
+open_connection(eval_environ_t env, char *urlstr, int *isfile, char **phost,
+ long timeout)
{
char *path = NULL;
short port = 0;
- mu_stream_t str = NULL;
+ mf_stream_t str = NULL;
int rc;
@@ -311,8 +441,8 @@ open_connection(eval_environ_t env, char *urlstr, int *isfile, char **phost)
if (port == 0) {
- rc = spamd_connect_socket(env, &str, path);
+ rc = spamd_connect_socket(env, &str, path, timeout);
*isfile = 1;
} else {
- rc = spamd_connect_tcp(env, &str, path, port);
+ rc = spamd_connect_tcp(env, &str, path, port, timeout);
*isfile = 0;
}
@@ -333,10 +463,10 @@ open_connection(eval_environ_t env, char *urlstr, int *isfile, char **phost)
MF_STATE(eom)
MF_CAPTURE
-MF_DEFUN(sa, NUMBER, STRING urlstr, NUMBER prec)
+MF_DEFUN(sa, NUMBER, STRING urlstr, NUMBER prec, OPTIONAL, NUMBER timeout)
{
mu_off_t msize;
size_t size;
mu_stream_t mstr = env_get_stream(env);
- mu_stream_t ostr;
+ mf_stream_t ostr;
signal_handler_fn handler;
char buffer[512];
@@ -348,5 +478,6 @@ MF_DEFUN(sa, NUMBER, STRING urlstr, NUMBER prec)
int isfile;
- ostr = open_connection(env, urlstr, &isfile, NULL);
+ ostr = open_connection(env, urlstr, &isfile, NULL,
+ MF_OPTVAL(timeout, 7200));
mu_stream_size(mstr, &msize);
spamd_send_command(ostr, "SYMBOLS SPAMC/1.2");
@@ -415,8 +546,9 @@ END
MF_STATE(eom)
MF_CAPTURE
-MF_DEFUN(clamav, NUMBER, STRING urlstr)
+MF_DEFUN(clamav, NUMBER, STRING urlstr, OPTIONAL, NUMBER timeout)
{
mu_stream_t mstr = env_get_stream(env);
- mu_stream_t cstr, dstr;
+ mf_stream_t cstr, dstr;
+ mu_stream_t str;
char buffer[512];
char *host;
@@ -426,6 +558,7 @@ MF_DEFUN(clamav, NUMBER, STRING urlstr)
char *p;
int isfile;
-
- cstr = open_connection(env, urlstr, &isfile, &host);
+
+ cstr = open_connection(env, urlstr, &isfile, &host,
+ MF_OPTVAL(timeout, 7200));
spamd_send_command(cstr, "STREAM");
@@ -440,6 +573,6 @@ MF_DEFUN(clamav, NUMBER, STRING urlstr)
if (!host)
host = strdup("127.0.0.1"); /* FIXME */
-
- rc = mu_tcp_stream_create(&dstr, host, port, 0);
+
+ rc = mu_tcp_stream_create(&str, host, port, 0);
free(host);
if (rc) {
@@ -450,8 +583,8 @@ MF_DEFUN(clamav, NUMBER, STRING urlstr)
}
- rc = mu_stream_open(dstr);
+ rc = mu_stream_open(str);
if (rc) {
spamd_destroy(&cstr);
- mu_stream_destroy(&dstr, mu_stream_get_owner(dstr));
+ mu_stream_destroy(&str, mu_stream_get_owner(str));
MF_THROW(mf_failure,
"mu_stream_open: %s",
@@ -460,6 +593,9 @@ MF_DEFUN(clamav, NUMBER, STRING urlstr)
handler = set_signal_handler(SIGPIPE, sigpipe_handler);
+ dstr = mf_stream_new (env, str, cstr->start, cstr->timeout);
spamd_send_stream(dstr, mstr);
spamd_shutdown(dstr, 0, SHUT_WR);
+ spamd_destroy(&dstr);
+
set_signal_handler(SIGPIPE, handler);
diff --git a/src/dnsbase.c b/src/dnsbase.c
index cf4b95a4..cbb658d0 100644
--- a/src/dnsbase.c
+++ b/src/dnsbase.c
@@ -248,5 +248,5 @@ struct loop_data {
char *hbuf; /* Return buffer */
size_t hbsize; /* Size of return buffer */
- size_t hbcount; /* ?? */
+ size_t hbcount; /* Number of items returned */
time_t ttl; /* TTL value */
@@ -419,5 +419,5 @@ cname_loop_body(struct loop_data *lp)
case T_TXT:
- if (lp->atype != T_ANY && lp->atype != type)
+ if (lp->atype != type)
continue;
l = cp[0];

Return to:

Send suggestions and report system problems to the System administrator.