diff options
author | Lennart Poettering <lennart@poettering.net> | 2017-12-14 19:47:46 +0100 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2017-12-14 19:47:46 +0100 |
commit | 673192494c129975b605541ee688143ecf2a6416 (patch) | |
tree | b67b544a5b9a0994bf4e2664a1968739bbebe77c | |
parent | fbd0b64f44a7bae678137d67ad5da202f6b8d809 (diff) | |
download | systemd-673192494c129975b605541ee688143ecf2a6416.tar.gz |
coccinelle: automatically rewrite memset() to zero() or memzero() where we can
We are pretty good at this already, hence only a single case is actually
found by this.
-rw-r--r-- | coccinelle/memzero.cocci | 30 | ||||
-rw-r--r-- | src/journal/journald-server.c | 2 |
2 files changed, 31 insertions, 1 deletions
diff --git a/coccinelle/memzero.cocci b/coccinelle/memzero.cocci new file mode 100644 index 0000000000..ebdc3f6a2a --- /dev/null +++ b/coccinelle/memzero.cocci @@ -0,0 +1,30 @@ +@@ +expression s; +@@ +- memset(&s, 0, sizeof(s)) ++ zero(s) +@@ +expression s; +@@ +- memset(s, 0, sizeof(*s)) ++ zero(*s) +@@ +expression s; +@@ +- bzero(&s, sizeof(s)) ++ zero(s) +@@ +expression s; +@@ +- bzero(s, sizeof(*s)) ++ zero(*s) +@@ +expression a, b; +@@ +- memset(a, 0, b) ++ memzero(a, b) +@@ +expression a, b; +@@ +- bzero(a, b) ++ memzero(a, b) diff --git a/src/journal/journald-server.c b/src/journal/journald-server.c index 48375f9c3e..d68e3a8d51 100644 --- a/src/journal/journald-server.c +++ b/src/journal/journald-server.c @@ -133,7 +133,7 @@ static int determine_path_usage(Server *s, const char *path, uint64_t *ret_used, } static void cache_space_invalidate(JournalStorageSpace *space) { - memset(space, 0, sizeof(*space)); + zero(*space); } static int cache_space_refresh(Server *s, JournalStorage *storage) { |