summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorGreg KH <gregkh@linuxfoundation.org>2014-02-02 07:15:23 -0500
committerAnthony G. Basile <blueness@gentoo.org>2014-02-02 07:15:23 -0500
commit8fe035158681d0324ff2be99a67bc0434da6faad (patch)
treec9a82a131e2597a62ed392355e39dfc7a888da9d /test
parent7eefa0ad0db267d99448fcddff2aebb4ebc33f6f (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). Signed-off-by: Anthony G. Basile <blueness@gentoo.org>
Diffstat (limited to 'test')
-rw-r--r--test/test-libudev.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/test/test-libudev.c b/test/test-libudev.c
index bb271e0c6e..71b981ca74 100644
--- a/test/test-libudev.c
+++ b/test/test-libudev.c
@@ -253,7 +253,7 @@ static int test_monitor(struct udev *udev) {
goto out;
}
- memset(&ep_udev, 0, sizeof(struct epoll_event));
+ memzero(&ep_udev, sizeof(struct epoll_event));
ep_udev.events = EPOLLIN;
ep_udev.data.fd = fd_udev;
if (epoll_ctl(fd_ep, EPOLL_CTL_ADD, fd_udev, &ep_udev) < 0) {
@@ -261,7 +261,7 @@ static int test_monitor(struct udev *udev) {
goto out;
}
- memset(&ep_stdin, 0, sizeof(struct epoll_event));
+ memzero(&ep_stdin, sizeof(struct epoll_event));
ep_stdin.events = EPOLLIN;
ep_stdin.data.fd = STDIN_FILENO;
if (epoll_ctl(fd_ep, EPOLL_CTL_ADD, STDIN_FILENO, &ep_stdin) < 0) {