mirror of
https://github.com/motioneye-project/motioneyeos.git
synced 2025-07-30 14:46:31 +00:00
tcpreplay: fix build errors when tcpdump (verbose) isn't enabled
Patch from Chris' pull request. Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
This commit is contained in:
parent
b304ff7f7d
commit
8476e5ad45
@ -0,0 +1,54 @@
|
|||||||
|
From 12d0d0876a9a9bf2818ba24ecb5646952b958c39 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Chris Packham <judge.packham@gmail.com>
|
||||||
|
Date: Wed, 22 Jan 2014 20:37:04 +1300
|
||||||
|
Subject: [PATCH] Fix build errors when ENABLE_VERBOSE is not set
|
||||||
|
|
||||||
|
Upstream: https://github.com/cpackham/tcpreplay/commit/12d0d0876a9a9
|
||||||
|
Fixes #59
|
||||||
|
|
||||||
|
When ENABLE_VERBOSE is not set some additional calls to tcpreplay_seterr
|
||||||
|
are made. When conforming to C99 variadic macros at least one argument is
|
||||||
|
required after the format string[1]. To avoid this problem update the
|
||||||
|
calls to tcpreplay_seterr to use a "%s" format string.
|
||||||
|
|
||||||
|
[1] - http://gcc.gnu.org/onlinedocs/cpp/Variadic-Macros.html
|
||||||
|
|
||||||
|
Signed-off-by: Chris Packham <judge.packham@gmail.com>
|
||||||
|
---
|
||||||
|
src/tcpreplay_api.c | 6 +++---
|
||||||
|
1 file changed, 3 insertions(+), 3 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/tcpreplay_api.c b/src/tcpreplay_api.c
|
||||||
|
index 6b81bcc..de67080 100644
|
||||||
|
--- a/src/tcpreplay_api.c
|
||||||
|
+++ b/src/tcpreplay_api.c
|
||||||
|
@@ -690,7 +690,7 @@ tcpreplay_set_verbose(tcpreplay_t *ctx, bool value)
|
||||||
|
ctx->options->verbose = value;
|
||||||
|
return 0;
|
||||||
|
#else
|
||||||
|
- tcpreplay_seterr(ctx, "verbose mode not supported");
|
||||||
|
+ tcpreplay_seterr(ctx, "%s", "verbose mode not supported");
|
||||||
|
return -1;
|
||||||
|
#endif
|
||||||
|
}
|
||||||
|
@@ -710,7 +710,7 @@ tcpreplay_set_tcpdump_args(tcpreplay_t *ctx, char *value)
|
||||||
|
ctx->options->tcpdump_args = safe_strdup(value);
|
||||||
|
return 0;
|
||||||
|
#else
|
||||||
|
- tcpreplay_seterr(ctx, "verbose mode not supported");
|
||||||
|
+ tcpreplay_seterr(ctx, "%s", "verbose mode not supported");
|
||||||
|
return -1;
|
||||||
|
#endif
|
||||||
|
}
|
||||||
|
@@ -731,7 +731,7 @@ tcpreplay_set_tcpdump(tcpreplay_t *ctx, tcpdump_t *value)
|
||||||
|
ctx->options->tcpdump = value;
|
||||||
|
return 0;
|
||||||
|
#else
|
||||||
|
- tcpreplay_seterr(ctx, "verbose mode not supported");
|
||||||
|
+ tcpreplay_seterr(ctx, "%s", "verbose mode not supported");
|
||||||
|
return -1;
|
||||||
|
#endif
|
||||||
|
}
|
||||||
|
--
|
||||||
|
1.8.5.2
|
||||||
|
|
Loading…
x
Reference in New Issue
Block a user