fix warning: this 'if' clause does not guard

HuO50 · HuO50 · commit b76eadfc067a · 2022-10-22T19:16:47.000+08:00
src/dmidecodemodule.c: In function 'dmidecode_get_xml':
src/dmidecodemodule.c:394:25: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
  394 |                         if(legacy_decode(opt-&gt;logdata, opt-&gt;type,
      |                         ^~
src/dmidecodemodule.c:397:33: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
  397 |                                 goto done;
      |                                 ^~~~
diff --git a/src/dmidecodemodule.c b/src/dmidecodemodule.c
@@ -392,9 +392,10 @@ int dmidecode_get_xml(options *opt, xmlNode* dmixml_n)
                         }
                 } else if(memcmp(buf + fp, "_DMI_", 5) == 0) {
                         if(legacy_decode(opt->logdata, opt->type,
-                                buf + fp, opt->devmem, 0, dmixml_n))
+                                buf + fp, opt->devmem, 0, dmixml_n)) {
                                 found++;
                                 goto done;
+                                }
                         }
                 }
 #endif

Original file line number	Diff line number	Diff line change
`@@ -392,9 +392,10 @@ int dmidecode_get_xml(options opt, xmlNode dmixml_n)`
`392`	`392`	`}`
`393`	`393`	`} else if(memcmp(buf + fp, "_DMI_", 5) == 0) {`
`394`	`394`	`if(legacy_decode(opt->logdata, opt->type,`
`395`		`- buf + fp, opt->devmem, 0, dmixml_n))`
	`395`	`+ buf + fp, opt->devmem, 0, dmixml_n)) {`
`396`	`396`	`found++;`
`397`	`397`	`goto done;`
	`398`	`+ }`
`398`	`399`	`}`
`399`	`400`	`}`
`400`	`401`	`#endif`