summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJim Meyering <jim@meyering.net>2007-07-20 15:38:19 +0000
committerJim Meyering <jim@meyering.net>2007-07-20 15:38:19 +0000
commitbbcd24607cc3e4a29abe39a93e0566554b9fb16a (patch)
tree6bc41c5f420e898a57593c7db9dff8ec279336f0
parente732de541e4010c975ace2c86e1896a1f282a9bd (diff)
downloadlvm2-bbcd24607cc3e4a29abe39a93e0566554b9fb16a.tar.gz
lvm2-bbcd24607cc3e4a29abe39a93e0566554b9fb16a.tar.xz
lvm2-bbcd24607cc3e4a29abe39a93e0566554b9fb16a.zip
Use gcc's printf attribute wherever possible.
* daemons/clvmd/clvmd.h (debuglog): Add __attribute__((printf)). * lib/config/config.c (_line_append): Likewise. * lib/misc/lvm-string.h (emit_to_buffer): Likewise.
-rw-r--r--WHATS_NEW1
-rw-r--r--daemons/clvmd/clvmd.h3
-rw-r--r--lib/config/config.c2
-rw-r--r--lib/misc/lvm-string.h3
4 files changed, 7 insertions, 2 deletions
diff --git a/WHATS_NEW b/WHATS_NEW
index 39b80008..6d7597e0 100644
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,4 +1,5 @@
Version 2.02.28 -
+ Use gcc's printf attribute wherever possible.
In _line_append, use "sizeof buf - 1" rather than equivalent "4095"
Introduce is_same_inode macro, now including a comparison of st_dev.
Don't leak a file descriptor in _lock_file(), when flock fails.
diff --git a/daemons/clvmd/clvmd.h b/daemons/clvmd/clvmd.h
index 8842936b..98c40cee 100644
--- a/daemons/clvmd/clvmd.h
+++ b/daemons/clvmd/clvmd.h
@@ -115,7 +115,8 @@ extern int add_client(struct local_client *new_client);
extern void clvmd_cluster_init_completed(void);
extern void process_message(struct local_client *client, const char *buf,
int len, const char *csid);
-extern void debuglog(const char *fmt, ... );
+extern void debuglog(const char *fmt, ... )
+ __attribute__ ((format(printf, 1, 2)));
int sync_lock(const char *resource, int mode, int flags, int *lockid);
int sync_unlock(const char *resource, int lockid);
diff --git a/lib/config/config.c b/lib/config/config.c
index bbfe7dfb..f15f46dc 100644
--- a/lib/config/config.c
+++ b/lib/config/config.c
@@ -361,6 +361,8 @@ static int _line_start(struct output_line *outline)
}
static int _line_append(struct output_line *outline, const char *fmt, ...)
+ __attribute__ ((format(printf, 2, 3)));
+static int _line_append(struct output_line *outline, const char *fmt, ...)
{
char buf[4096];
va_list ap;
diff --git a/lib/misc/lvm-string.h b/lib/misc/lvm-string.h
index c5a87811..12e7957d 100644
--- a/lib/misc/lvm-string.h
+++ b/lib/misc/lvm-string.h
@@ -23,7 +23,8 @@
struct pool;
-int emit_to_buffer(char **buffer, size_t *size, const char *fmt, ...);
+int emit_to_buffer(char **buffer, size_t *size, const char *fmt, ...)
+ __attribute__ ((format(printf, 3, 4)));
char *build_dm_name(struct dm_pool *mem, const char *vg,
const char *lv, const char *layer);