Re: [dpdk-dev] [PATCH 0/3] Various patches

[Thomas Monjalon <thomas.monjalon@6wind.com>]

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@6wind.com>

Applied for version 1.7.0.

Thanks for these difficult catches, especially the mempool/get_gcd one!
-- 
Thomas