From: Frank Lichtenheld <[email protected]> len is already int, no need to throw another type into the mix.
Change-Id: Ida8e587a095bdfb821ee0a4783633f026d310476 Signed-off-by: Frank Lichtenheld <[email protected]> Acked-by: Arne Schwabe <[email protected]> Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1494 --- This change was reviewed on Gerrit and approved by at least one developer. I request to merge it to master. Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1494 This mail reflects revision 5 of this Change. Acked-by according to Gerrit (reflected above): Arne Schwabe <[email protected]> diff --git a/src/openvpn/argv.c b/src/openvpn/argv.c index b5d9603..6a5d92e 100644 --- a/src/openvpn/argv.c +++ b/src/openvpn/argv.c @@ -263,11 +263,6 @@ gc_free(&gc); } -#if defined(__GNUC__) || defined(__clang__) -#pragma GCC diagnostic push -#pragma GCC diagnostic ignored "-Wsign-compare" -#endif - /** * Prepares argv format string for further processing * @@ -389,7 +384,7 @@ * Do the actual vsnprintf() operation, which expands the format * string with the provided arguments. */ - size_t size = len + 1; + int size = len + 1; char *buf = gc_malloc(size, false, &argres->gc); len = vsnprintf(buf, size, f, arglist); if (len < 0 || len >= size) @@ -423,10 +418,6 @@ return res; } -#if defined(__GNUC__) || defined(__clang__) -#pragma GCC diagnostic pop -#endif - /** * printf() variant which populates a struct argv. It processes the * format string with the provided arguments. For each space separator found _______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
