summaryrefslogtreecommitdiffabout
authorSergey Poznyakoff <gray@gnu.org.ua>2014-05-29 10:22:59 (GMT)
committer Sergey Poznyakoff <gray@gnu.org.ua>2014-05-29 14:04:56 (GMT)
commit2035536f831b0ce54c28c2a0b405febb113e0fab (patch) (side-by-side diff)
tree2408086b3eae646006a39e0b78892e2838cb2f35
parent35e2816614ab2a1aa585b1ae2547683a69ed9d22 (diff)
downloadvmod-binlog-2035536f831b0ce54c28c2a0b405febb113e0fab.tar.gz
vmod-binlog-2035536f831b0ce54c28c2a0b405febb113e0fab.tar.bz2
Further thread-safety fixes.
* NEWS: Version 1.0.90 * configure.ac: Likewise. * src/binlog.c (binlog_env) <inst_head>: New member. (binlog_env_init): Initialize inst_head and use it instead of conf->inst_head. (env_free): Free inst_head. * src/pack.c (packdup): New function. * src/pack.h (packdup): New proto.
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--NEWS4
-rw-r--r--configure.ac4
-rw-r--r--src/binlog.c10
-rw-r--r--src/binlogsel.c2
-rw-r--r--src/pack.c25
-rw-r--r--src/pack.h1
6 files changed, 39 insertions, 7 deletions
diff --git a/NEWS b/NEWS
index 4546558..e8b0f8b 100644
--- a/NEWS
+++ b/NEWS
@@ -1,9 +1,11 @@
-Vmod-binlog NEWS -- history of user-visible changes. 2013-10-19
+Vmod-binlog NEWS -- history of user-visible changes. 2014-05-29
Copyright (C) 2013 Sergey Poznyakoff
See the end of file for copying conditions.
Please send Vmod-binlog bug reports to <gray@gnu.org>
+Version 1,0.90 (Git)
+
Version 1.0, 2013-10-19
Initial release.
diff --git a/configure.ac b/configure.ac
index 692b052..3d80940 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,5 +1,5 @@
# This file is part of vmod-binlog
-# Copyright (C) 2013 Sergey Poznyakoff
+# Copyright (C) 2013, 2014 Sergey Poznyakoff
#
# Vmod-binlog is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
@@ -14,7 +14,7 @@
# You should have received a copy of the GNU General Public License
# along with vmod-binlog. If not, see <http://www.gnu.org/licenses/>.
AC_PREREQ(2.69)
-AC_INIT([vmod-binlog], 1.0, [gray@gnu.org])
+AC_INIT([vmod-binlog], 1.0.90, [gray@gnu.org])
AC_CONFIG_AUX_DIR([build-aux])
AC_CONFIG_MACRO_DIR([m4])
AC_CONFIG_SRCDIR(src/binlog.c)
diff --git a/src/binlog.c b/src/binlog.c
index 4c3f40e..45299f2 100644
--- a/src/binlog.c
+++ b/src/binlog.c
@@ -1,5 +1,5 @@
/* This file is part of vmod-binlog
- Copyright (C) 2013 Sergey Poznyakoff
+ Copyright (C) 2013, 2014 Sergey Poznyakoff
Vmod-binlog is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
@@ -72,6 +72,7 @@ struct binlog_env {
enum binlog_state state; /* binlog machine state */
struct binlog_config *conf;
time_t timestamp; /* timestamp for the entry being built */
+ struct packinst *inst_head; /* compiled format */
struct packinst *inst_cur; /* current instruction */
struct packenv *env; /* pack environment */
};
@@ -121,12 +122,14 @@ void
binlog_env_init(struct binlog_env *ep, struct binlog_config *conf, time_t ts)
{
if (!ep->env) {
- ep->env = packenv_create(packsize(conf->inst_head));
+ ep->inst_head = packdup(conf->inst_head);
+ AN(ep->inst_head);
+ ep->env = packenv_create(packsize(ep->inst_head));
AN(ep->env);
}
packenv_init(ep->env);
ep->state = state_start;
- ep->inst_cur = packinit(conf->inst_head);
+ ep->inst_cur = packinit(ep->inst_head);
ep->timestamp = ts;
}
@@ -247,6 +250,7 @@ static void
env_free(void *f)
{
struct binlog_env *ep = f;
+ packfree(ep->inst_head);
packenv_free(ep->env);
free(ep);
}
diff --git a/src/binlogsel.c b/src/binlogsel.c
index fad0881..da04636 100644
--- a/src/binlogsel.c
+++ b/src/binlogsel.c
@@ -1,5 +1,5 @@
/* This file is part of vmod-binlog
- Copyright (C) 2013 Sergey Poznyakoff
+ Copyright (C) 2013, 2014 Sergey Poznyakoff
Vmod-binlog is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
diff --git a/src/pack.c b/src/pack.c
index 9e9e0ed..6c21c89 100644
--- a/src/pack.c
+++ b/src/pack.c
@@ -714,6 +714,31 @@ packcomp(const char *s, char **endp)
return head;
}
+struct packinst *
+packdup(struct packinst *src)
+{
+ struct packinst *head = NULL, *tail = NULL, *p;
+
+ for (; src; src = src->next) {
+ p = malloc(sizeof(*p));
+ if (!p) {
+ packfree(head);
+ errno = ENOMEM;
+ return NULL;
+ }
+ p->next = NULL;
+ p->spec = src->spec;
+ p->rep = src->rep;
+ p->cur = src->cur;
+ if (tail)
+ tail->next = p;
+ else
+ head = p;
+ tail = p;
+ }
+ return head;
+}
+
void
packfree(struct packinst *pi)
{
diff --git a/src/pack.h b/src/pack.h
index c134878..d32a8fa 100644
--- a/src/pack.h
+++ b/src/pack.h
@@ -29,6 +29,7 @@ struct packenv {
struct packinst;
struct packinst *packcomp(const char *s, char **endp);
+struct packinst *packdup(struct packinst *s);
void packfree(struct packinst *pi);
struct packinst *packinit(struct packinst *pi);
void packin(struct packinst *pi, struct packenv *env);

Return to:

Send suggestions and report system problems to the System administrator.