Fix: use assignment-suppression for unused sscanf arguments
authorJérémie Galarneau <jeremie.galarneau@efficios.com>
Wed, 12 Dec 2018 02:26:42 +0000 (21:26 -0500)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Wed, 12 Dec 2018 02:27:49 +0000 (21:27 -0500)
This removes the conversion of elements parsed by sscanf() which
are not used anyhow and eliminates a warning on x86 builds
(%lu used on size_t).

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
src/bin/lttng/commands/enable_events.c

index f7c5553da55b211383e31fb0fe92c3c6083693d1..b4e6320c52a84619888b344b7037107c1aed8fc2 100644 (file)
@@ -305,12 +305,10 @@ end:
  */
 static int warn_userspace_probe_syntax(const char *symbol)
 {
  */
 static int warn_userspace_probe_syntax(const char *symbol)
 {
-       unsigned long addr = 0;
-       size_t offset = 0;
        int ret;
 
        /* Check if the symbol field is an hex address. */
        int ret;
 
        /* Check if the symbol field is an hex address. */
-       ret = sscanf(symbol, "0x%lx", &addr);
+       ret = sscanf(symbol, "0x%*x");
        if (ret > 0) {
                /* If there is a match, print a warning and return an error. */
                ERR("Userspace probe on address not supported yet.");
        if (ret > 0) {
                /* If there is a match, print a warning and return an error. */
                ERR("Userspace probe on address not supported yet.");
@@ -319,7 +317,7 @@ static int warn_userspace_probe_syntax(const char *symbol)
        }
 
        /* Check if the symbol field is an decimal address. */
        }
 
        /* Check if the symbol field is an decimal address. */
-       ret = sscanf(symbol, "%lu", &addr);
+       ret = sscanf(symbol, "%*u");
        if (ret > 0) {
                /* If there is a match, print a warning and return an error. */
                ERR("Userspace probe on address not supported yet.");
        if (ret > 0) {
                /* If there is a match, print a warning and return an error. */
                ERR("Userspace probe on address not supported yet.");
@@ -328,7 +326,7 @@ static int warn_userspace_probe_syntax(const char *symbol)
        }
 
        /* Check if the symbol field is symbol+hex_offset. */
        }
 
        /* Check if the symbol field is symbol+hex_offset. */
-       ret = sscanf(symbol, "%*[^+]+0x%lx", &offset);
+       ret = sscanf(symbol, "%*[^+]+0x%*x");
        if (ret > 0) {
                /* If there is a match, print a warning and return an error. */
                ERR("Userspace probe on symbol+offset not supported yet.");
        if (ret > 0) {
                /* If there is a match, print a warning and return an error. */
                ERR("Userspace probe on symbol+offset not supported yet.");
@@ -337,7 +335,7 @@ static int warn_userspace_probe_syntax(const char *symbol)
        }
 
        /* Check if the symbol field is symbol+decimal_offset. */
        }
 
        /* Check if the symbol field is symbol+decimal_offset. */
-       ret = sscanf(symbol, "%*[^+]+%lu", &offset);
+       ret = sscanf(symbol, "%*[^+]+%*u");
        if (ret > 0) {
                /* If there is a match, print a warning and return an error. */
                ERR("Userspace probe on symbol+offset not supported yet.");
        if (ret > 0) {
                /* If there is a match, print a warning and return an error. */
                ERR("Userspace probe on symbol+offset not supported yet.");
This page took 0.03833 seconds and 5 git commands to generate.