projects
/
collectd.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
ff9e068
)
write_http plugin: Sanity check arguments to wh_write_command().
author
Florian Forster
<octo@collectd.org>
Sun, 14 Aug 2016 18:47:15 +0000
(20:47 +0200)
committer
Florian Forster
<octo@collectd.org>
Sun, 14 Aug 2016 18:47:15 +0000
(20:47 +0200)
Issue: #1835
src/write_http.c
patch
|
blob
|
history
diff --git
a/src/write_http.c
b/src/write_http.c
index
95132cf
..
41615d3
100644
(file)
--- a/
src/write_http.c
+++ b/
src/write_http.c
@@
-362,7
+362,11
@@
static int wh_write_command (const data_set_t *ds, const value_list_t *vl, /* {{
int status;
- if (0 != strcmp (ds->type, vl->type)) {
+ /* sanity checks, primarily to make static analyzers happy. */
+ if ((cb == NULL) || (cb->send_buffer == NULL))
+ return -1;
+
+ if (strcmp (ds->type, vl->type) == 0) {
ERROR ("write_http plugin: DS type does not match "
"value list type");
return -1;