summaryrefslogtreecommitdiffabout
authorSergey Poznyakoff <gray@gnu.org.ua>2016-01-15 12:33:18 (GMT)
committer Sergey Poznyakoff <gray@gnu.org.ua>2016-01-15 12:33:18 (GMT)
commitc90ba2a863a446b51b9ee960cead69a4b2884045 (patch) (side-by-side diff)
treea6c36a8a15fd00f49dd3b0b46ca619563571e1f0
parent746f6355a4595cc4bbcffc89a3e42a082d757759 (diff)
downloadvmod-variable-c90ba2a863a446b51b9ee960cead69a4b2884045.tar.gz
vmod-variable-c90ba2a863a446b51b9ee960cead69a4b2884045.tar.bz2
Minor fixes.
Remove unused variables, shut gcc warnings. * src/variable.c (get_symtab): Bail out if cache fd is less than 0.
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--src/variable.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/variable.c b/src/variable.c
index bd79c3b..e6bad1a 100644
--- a/src/variable.c
+++ b/src/variable.c
@@ -388,2 +388,4 @@ get_symtab(VARIABLE_CTX ctx)
int fd = ctx->req->sp->fd;
+
+ assert(fd >= 0);
AZ(pthread_mutex_lock(&symtab_mtx));
@@ -730,3 +732,2 @@ vmod_regset(VARIABLE_CTX ctx, VCL_STRING vars, VCL_STRING rxs,
union value value;
- char *p;
int ovsize;
@@ -864,3 +865,3 @@ xdecode(char *s)
if (*s == '%' && hex2ul(s[1]) != -1 && hex2ul(s[2]) != -1) {
- *d++ = hex2ul(s[1]) << 4 + hex2ul(s[2]);
+ *d++ = (hex2ul(s[1]) << 4) + hex2ul(s[2]);
s += 3;
@@ -900,3 +901,2 @@ vmod_queryset(VARIABLE_CTX ctx, VCL_STRING vars, VCL_STRING query)
size_t count = 0;
- size_t n;
const char *v = vars;
@@ -965,3 +965,3 @@ vmod_queryset(VARIABLE_CTX ctx, VCL_STRING vars, VCL_STRING query)
define_param(vt, def, valptr, vallen);
- free(def);
+ vardef_free(def);
}

Return to:

Send suggestions and report system problems to the System administrator.