diff options
author | Greg KH <gregkh@linuxfoundation.org> | 2014-01-31 06:51:32 +0100 |
---|---|---|
committer | Tom Gundersen <teg@jklm.no> | 2014-01-31 11:55:01 +0100 |
commit | 29804cc1e0f37ee34301530fd7f1eb8550be464e (patch) | |
tree | ba287b984b216279bb342579a9d1039383ca3a39 /src/python-systemd | |
parent | b5cfa7408c4cb68e9bb232fc34b07fd03c915617 (diff) |
use memzero(foo, length); for all memset(foo, 0, length); calls
In trying to track down a stupid linker bug, I noticed a bunch of
memset() calls that should be using memzero() to make it more "obvious"
that the options are correct (i.e. 0 is not the length, but the data to
set). So fix up all current calls to memset(foo, 0, length) to
memzero(foo, length).
Diffstat (limited to 'src/python-systemd')
-rw-r--r-- | src/python-systemd/_journal.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/python-systemd/_journal.c b/src/python-systemd/_journal.c index f8e0b4f7f2..669c22ce5e 100644 --- a/src/python-systemd/_journal.c +++ b/src/python-systemd/_journal.c @@ -22,6 +22,7 @@ #include <Python.h> #include <alloca.h> +#include "util.h" #define SD_JOURNAL_SUPPRESS_LOCATION #include <systemd/sd-journal.h> @@ -41,7 +42,7 @@ static PyObject *journal_sendv(PyObject *self, PyObject *args) { /* Allocate an array for the argument strings */ argc = PyTuple_Size(args); encoded = alloca(argc * sizeof(PyObject*)); - memset(encoded, 0, argc * sizeof(PyObject*)); + memzero(encoded, argc * sizeof(PyObject*)); /* Allocate sufficient iovector space for the arguments. */ iov = alloca(argc * sizeof(struct iovec)); |