diff options
-rw-r--r-- | ChangeLog | 3 | ||||
-rw-r--r-- | src/bi_sa.m4 | 218 | ||||
-rw-r--r-- | src/dnsbase.c | 4 |
3 files changed, 182 insertions, 43 deletions
@@ -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]; |