diff options
| author | Pranith Kumar K <pkarampu@redhat.com> | 2016-10-18 15:16:17 +0530 |
|---|---|---|
| committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2016-11-18 01:56:39 -0800 |
| commit | 5310be8838f8db748a698bd3a98f8d00a4114e65 (patch) | |
| tree | dcfdf14c322d8731cec0eafc7aa24bf1abbf6629 /libglusterfs | |
| parent | 2f70f47979d204d5ee29811f5f5fa61d0a52c3b1 (diff) | |
events: Add FMT_WARN for gf_event
Raghavendra G found that posix is trying to print %s
but passing an int when HEALTH_CHECK fails in posix.
These are the kind of bugs that should be caught
at compilation itself.
Also fixed the problematic gf_event() callers.
BUG: 1386097
Change-Id: Id7bd6d9a9690237cec3ca1aefa2aac085e8a1270
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/15671
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'libglusterfs')
| -rw-r--r-- | libglusterfs/src/events.c | 2 | ||||
| -rw-r--r-- | libglusterfs/src/events.h | 10 |
2 files changed, 9 insertions, 3 deletions
diff --git a/libglusterfs/src/events.c b/libglusterfs/src/events.c index b7b513eb39a..6b3a73d8ae4 100644 --- a/libglusterfs/src/events.c +++ b/libglusterfs/src/events.c @@ -32,7 +32,7 @@ int -gf_event (eventtypes_t event, char *fmt, ...) +_gf_event (eventtypes_t event, char *fmt, ...) { int ret = 0; int sock = -1; diff --git a/libglusterfs/src/events.h b/libglusterfs/src/events.h index 6b280a0066f..e059bc9c4d1 100644 --- a/libglusterfs/src/events.h +++ b/libglusterfs/src/events.h @@ -15,13 +15,19 @@ #ifdef USE_EVENTS int -gf_event (eventtypes_t event, char *fmt, ...); +_gf_event (eventtypes_t event, char *fmt, ...); #else static inline int -gf_event (eventtypes_t event, char *fmt, ...) +_gf_event (eventtypes_t event, char *fmt, ...) { return 0; } #endif /* USE_EVENTS */ +#define gf_event(event, fmt...) \ + do { \ + FMT_WARN(fmt); \ + _gf_event (event, ##fmt); \ + } while (0) + #endif /* __EVENTS_H__ */ |