diff options
| author | christos <christos@NetBSD.org> | 2014-03-27 18:27:34 +0000 |
|---|---|---|
| committer | christos <christos@NetBSD.org> | 2014-03-27 18:27:34 +0000 |
| commit | 9b7d56537e7e431b68ec800eff4fc0d3033a0708 (patch) | |
| tree | f4da25f96de1aca296ff8beb2a2827718559b696 /sys/dev | |
| parent | 19d51526308c582d9e7f13c7d33bea4c0394edff (diff) | |
remove a bunch of repetitive code by introducing filemon_printf.
Diffstat (limited to 'sys/dev')
| -rw-r--r-- | sys/dev/filemon/filemon.c | 26 | ||||
| -rw-r--r-- | sys/dev/filemon/filemon.h | 3 | ||||
| -rw-r--r-- | sys/dev/filemon/filemon_wrapper.c | 87 |
3 files changed, 33 insertions, 83 deletions
diff --git a/sys/dev/filemon/filemon.c b/sys/dev/filemon/filemon.c index 596bbe1a670..a13c55a1261 100644 --- a/sys/dev/filemon/filemon.c +++ b/sys/dev/filemon/filemon.c @@ -24,7 +24,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: filemon.c,v 1.6 2014/03/16 05:20:27 dholland Exp $"); +__KERNEL_RCSID(0, "$NetBSD: filemon.c,v 1.7 2014/03/27 18:27:34 christos Exp $"); #include <sys/param.h> #include <sys/kernel.h> @@ -121,16 +121,26 @@ filemon_output(struct filemon * filemon, char *msg, size_t len) &auio, curlwp->l_cred, FOF_UPDATE_OFFSET); } +void +filemon_printf(struct filemon *filemon, const char *fmt, ...) +{ + size_t len; + va_list ap; + + va_start(ap, fmt); + len = vsnprintf(filemon->fm_msgbufr, sizeof(filemon->fm_msgbufr), + fmt, ap); + va_end(ap); + if (len > sizeof(filemon->fm_msgbufr)) + len = sizeof(filemon->fm_msgbufr); + filemon_output(filemon, filemon->fm_msgbufr, len); +} + static void filemon_comment(struct filemon * filemon) { - int len; - - len = snprintf(filemon->fm_msgbufr, sizeof(filemon->fm_msgbufr), - "# filemon version %d\n# Target pid %d\nV %d\n", - FILEMON_VERSION, curproc->p_pid, FILEMON_VERSION); - - filemon_output(filemon, filemon->fm_msgbufr, len); + filemon_printf(filemon, "# filemon version %d\n# Target pid %d\nV %d\n", + FILEMON_VERSION, curproc->p_pid, FILEMON_VERSION); } diff --git a/sys/dev/filemon/filemon.h b/sys/dev/filemon/filemon.h index 95f445e51d5..560175d6df9 100644 --- a/sys/dev/filemon/filemon.h +++ b/sys/dev/filemon/filemon.h @@ -1,4 +1,4 @@ -/* $NetBSD: filemon.h,v 1.4 2012/11/19 22:20:10 sjg Exp $ */ +/* $NetBSD: filemon.h,v 1.5 2014/03/27 18:27:34 christos Exp $ */ /* * Copyright (c) 2010, Juniper Networks, Inc. * @@ -49,6 +49,7 @@ struct filemon * filemon_lookup(struct proc *); void filemon_output(struct filemon *, char *, size_t); void filemon_wrapper_install(void); int filemon_wrapper_deinstall(void); +void filemon_printf(struct filemon *, const char *, ...) __printflike(2, 3); #endif #endif diff --git a/sys/dev/filemon/filemon_wrapper.c b/sys/dev/filemon/filemon_wrapper.c index 712e9009404..4d0419928b1 100644 --- a/sys/dev/filemon/filemon_wrapper.c +++ b/sys/dev/filemon/filemon_wrapper.c @@ -24,7 +24,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: filemon_wrapper.c,v 1.4 2012/11/19 22:20:10 sjg Exp $"); +__KERNEL_RCSID(0, "$NetBSD: filemon_wrapper.c,v 1.5 2014/03/27 18:27:34 christos Exp $"); #include <sys/param.h> #include <sys/types.h> @@ -44,7 +44,6 @@ filemon_wrapper_chdir(struct lwp * l, const struct sys_chdir_args * uap, int ret; int error; size_t done; - size_t len; struct filemon *filemon; if ((ret = sys_chdir(l, uap, retval)) == 0) { @@ -55,13 +54,9 @@ filemon_wrapper_chdir(struct lwp * l, const struct sys_chdir_args * uap, error = copyinstr(SCARG(uap, path), filemon->fm_fname1, sizeof(filemon->fm_fname1), &done); if (error == 0) { - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), + filemon_printf(filemon, "C %d %s\n", curproc->p_pid, filemon->fm_fname1); - - filemon_output(filemon, filemon->fm_msgbufr, - len); } rw_exit(&filemon->fm_mtx); } @@ -77,7 +72,6 @@ filemon_wrapper_execve(struct lwp * l, struct sys_execve_args * uap, int ret; int error; size_t done; - size_t len; struct filemon *filemon; error = copyinstr(SCARG(uap, path), fname, sizeof(fname), &done); @@ -86,12 +80,8 @@ filemon_wrapper_execve(struct lwp * l, struct sys_execve_args * uap, filemon = filemon_lookup(curproc); if (filemon) { - - len = snprintf(filemon->fm_msgbufr, sizeof(filemon->fm_msgbufr), - "E %d %s\n", + filemon_printf(filemon, "E %d %s\n", curproc->p_pid, fname); - - filemon_output(filemon, filemon->fm_msgbufr, len); rw_exit(&filemon->fm_mtx); } } @@ -103,20 +93,14 @@ static int filemon_wrapper_fork(struct lwp * l, const void *v, register_t * retval) { int ret; - size_t len; struct filemon *filemon; if ((ret = sys_fork(l, v, retval)) == 0) { filemon = filemon_lookup(curproc); if (filemon) { - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), - "F %d %ld\n", + filemon_printf(filemon, "F %d %ld\n", curproc->p_pid, (long) retval[0]); - - filemon_output(filemon, filemon->fm_msgbufr, len); - rw_exit(&filemon->fm_mtx); } } @@ -127,20 +111,14 @@ static int filemon_wrapper_vfork(struct lwp * l, const void *v, register_t * retval) { int ret; - size_t len; struct filemon *filemon; if ((ret = sys_vfork(l, v, retval)) == 0) { filemon = filemon_lookup(curproc); if (filemon) { - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), - "F %d %ld\n", + filemon_printf(filemon, "F %d %ld\n", curproc->p_pid, (long) retval[0]); - - filemon_output(filemon, filemon->fm_msgbufr, len); - rw_exit(&filemon->fm_mtx); } } @@ -154,7 +132,6 @@ filemon_wrapper_open(struct lwp * l, struct sys_open_args * uap, int ret; int error; size_t done; - size_t len; struct filemon *filemon; if ((ret = sys_open(l, uap, retval)) == 0) { @@ -166,23 +143,15 @@ filemon_wrapper_open(struct lwp * l, struct sys_open_args * uap, if (error == 0) { if (SCARG(uap, flags) & O_RDWR) { /* we want a separate R record */ - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), + filemon_printf(filemon, "R %d %s\n", curproc->p_pid, filemon->fm_fname1); - - filemon_output(filemon, - filemon->fm_msgbufr, len); } - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), + filemon_printf(filemon, "%c %d %s\n", (SCARG(uap, flags) & O_ACCMODE) ? 'W' : 'R', curproc->p_pid, filemon->fm_fname1); - - filemon_output(filemon, filemon->fm_msgbufr, - len); } rw_exit(&filemon->fm_mtx); } @@ -197,7 +166,6 @@ filemon_wrapper_rename(struct lwp * l, struct sys_rename_args * uap, int ret; int error; size_t done; - size_t len; struct filemon *filemon; if ((ret = sys_rename(l, uap, retval)) == 0) { @@ -211,14 +179,10 @@ filemon_wrapper_rename(struct lwp * l, struct sys_rename_args * uap, filemon->fm_fname2, sizeof(filemon->fm_fname2), &done); if (error == 0) { - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), + filemon_printf(filemon, "M %d '%s' '%s'\n", curproc->p_pid, filemon->fm_fname1, filemon->fm_fname2); - - filemon_output(filemon, filemon->fm_msgbufr, - len); } rw_exit(&filemon->fm_mtx); } @@ -233,7 +197,6 @@ filemon_wrapper_link(struct lwp * l, struct sys_link_args * uap, int ret; int error; size_t done; - size_t len; struct filemon *filemon; if ((ret = sys_link(l, uap, retval)) == 0) { @@ -248,13 +211,9 @@ filemon_wrapper_link(struct lwp * l, struct sys_link_args * uap, filemon->fm_fname2, sizeof(filemon->fm_fname2), &done); if (error == 0) { - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), "L %d '%s' '%s'\n", + filemon_printf(filemon, "L %d '%s' '%s'\n", curproc->p_pid, filemon->fm_fname1, filemon->fm_fname2); - - filemon_output(filemon, filemon->fm_msgbufr, - len); } rw_exit(&filemon->fm_mtx); } @@ -269,7 +228,6 @@ filemon_wrapper_symlink(struct lwp * l, struct sys_symlink_args * uap, int ret; int error; size_t done; - size_t len; struct filemon *filemon; if ((ret = sys_symlink(l, uap, retval)) == 0) { @@ -284,14 +242,9 @@ filemon_wrapper_symlink(struct lwp * l, struct sys_symlink_args * uap, filemon->fm_fname2, sizeof(filemon->fm_fname2), &done); if (error == 0) { - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), - "L %d '%s' '%s'\n", + filemon_printf(filemon, "L %d '%s' '%s'\n", curproc->p_pid, filemon->fm_fname1, filemon->fm_fname2); - - filemon_output(filemon, filemon->fm_msgbufr, - len); } rw_exit(&filemon->fm_mtx); } @@ -304,24 +257,16 @@ static void filemon_wrapper_sys_exit(struct lwp * l, struct sys_exit_args * uap, register_t * retval) { - size_t len; struct filemon *filemon; filemon = filemon_lookup(curproc); if (filemon) { - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), "X %d %d\n", + filemon_printf(filemon, "X %d %d\n", curproc->p_pid, SCARG(uap, rval)); - - filemon_output(filemon, filemon->fm_msgbufr, len); - /* Check if the monitored process is about to exit. */ if (filemon->fm_pid == curproc->p_pid) { - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), "# Bye bye\n"); - - filemon_output(filemon, filemon->fm_msgbufr, len); + filemon_printf(filemon, "# Bye bye\n"); } rw_exit(&filemon->fm_mtx); } @@ -335,7 +280,6 @@ filemon_wrapper_unlink(struct lwp * l, struct sys_unlink_args * uap, int ret; int error; size_t done; - size_t len; struct filemon *filemon; if ((ret = sys_unlink(l, uap, retval)) == 0) { @@ -346,13 +290,8 @@ filemon_wrapper_unlink(struct lwp * l, struct sys_unlink_args * uap, filemon->fm_fname1, sizeof(filemon->fm_fname1), &done); if (error == 0) { - len = snprintf(filemon->fm_msgbufr, - sizeof(filemon->fm_msgbufr), - "D %d %s\n", + filemon_printf(filemon, "D %d %s\n", curproc->p_pid, filemon->fm_fname1); - - filemon_output(filemon, filemon->fm_msgbufr, - len); } rw_exit(&filemon->fm_mtx); } |
