2014-05-12 17:35, Julien Cretin: > This patch set contains three unrelated patches found by running > TrustInSoft Analyzer [1] on some parts of the testpmd application: > > - The first patch fixes a minor signed overflow in a constant > expression of testpmd. > > - The second patch is not a fix and concerns a suspicious loop > condition in optimize_object_size. > > - The third (and last) patch fixes sign mismatches for some printf > arguments. > > [1] TrustInSoft Analyzer (http://trust-in-soft.com) is a static > analyzer. And as such, it gives information about the execution of a > source code without actually running it. However, unlike other static > analysis tools, it has the particularity of being correct. This means > that it does not remain silent when a run-time error may happen in the > range of the analysis. In other words, it gives information about all > possible executions (defined by the analysis) of a source code without > actually running it. > > Julien Cretin (3): > app/testpmd: fix minor signed overflow in a constant > mem: remove redundant check in optimize_object_size > app/testpmd: fix incompatible sign for printf arguments
Acked-by: Thomas Monjalon <thomas.monjalon at 6wind.com> Applied for version 1.7.0. Thanks for these difficult catches, especially the mempool/get_gcd one! -- Thomas
