diff mbox

test:performance:initialize undefined values

Message ID 1458901858-25237-1-git-send-email-balakrishna.garapati@linaro.org
State Accepted
Commit 03d383af4f1c19f8ed229e358942992f8852c813
Headers show

Commit Message

Balakrishna Garapati March 25, 2016, 10:30 a.m. UTC
Signed-off-by: Balakrishna Garapati <balakrishna.garapati@linaro.org>
---
 v1:fix for valgrind erros from crypto test
    https://bugs.linaro.org/show_bug.cgi?id=2141

 test/performance/odp_crypto.c | 2 ++
 1 file changed, 2 insertions(+)

--
1.9.1

Comments

Bill Fischofer March 25, 2016, 3:53 p.m. UTC | #1
On Fri, Mar 25, 2016 at 5:30 AM, Balakrishna Garapati <
balakrishna.garapati@linaro.org> wrote:

> Signed-off-by: Balakrishna Garapati <balakrishna.garapati@linaro.org>

>


Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org>



> ---

>  v1:fix for valgrind erros from crypto test

>     https://bugs.linaro.org/show_bug.cgi?id=2141

>

>  test/performance/odp_crypto.c | 2 ++

>  1 file changed, 2 insertions(+)

>

> diff --git a/test/performance/odp_crypto.c b/test/performance/odp_crypto.c

> index 5fd339d..6bd5cf5 100644

> --- a/test/performance/odp_crypto.c

> +++ b/test/performance/odp_crypto.c

> @@ -726,6 +726,8 @@ int main(int argc, char *argv[])

>         odp_cpumask_t cpumask;

>         char cpumaskstr[ODP_CPUMASK_STR_SIZE];

>

> +       memset(&cargs, 0, sizeof(cargs));

> +

>         /* Parse and store the application arguments */

>         parse_args(argc, argv, &cargs);

>

> --

> 1.9.1

>

> _______________________________________________

> lng-odp mailing list

> lng-odp@lists.linaro.org

> https://lists.linaro.org/mailman/listinfo/lng-odp

>
Maxim Uvarov March 28, 2016, 5:30 p.m. UTC | #2
patch was merged,
Maxim.

On 03/25/16 18:53, Bill Fischofer wrote:
>
>
> On Fri, Mar 25, 2016 at 5:30 AM, Balakrishna Garapati 
> <balakrishna.garapati@linaro.org 
> <mailto:balakrishna.garapati@linaro.org>> wrote:
>
>     Signed-off-by: Balakrishna Garapati
>     <balakrishna.garapati@linaro.org
>     <mailto:balakrishna.garapati@linaro.org>>
>
>
> Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org 
> <mailto:bill.fischofer@linaro.org>>
>
>     ---
>      v1:fix for valgrind erros from crypto test
>     https://bugs.linaro.org/show_bug.cgi?id=2141
>
>      test/performance/odp_crypto.c | 2 ++
>      1 file changed, 2 insertions(+)
>
>     diff --git a/test/performance/odp_crypto.c
>     b/test/performance/odp_crypto.c
>     index 5fd339d..6bd5cf5 100644
>     --- a/test/performance/odp_crypto.c
>     +++ b/test/performance/odp_crypto.c
>     @@ -726,6 +726,8 @@ int main(int argc, char *argv[])
>             odp_cpumask_t cpumask;
>             char cpumaskstr[ODP_CPUMASK_STR_SIZE];
>
>     +       memset(&cargs, 0, sizeof(cargs));
>     +
>             /* Parse and store the application arguments */
>             parse_args(argc, argv, &cargs);
>
>     --
>     1.9.1
>
>     _______________________________________________
>     lng-odp mailing list
>     lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
>     https://lists.linaro.org/mailman/listinfo/lng-odp
>
>
diff mbox

Patch

diff --git a/test/performance/odp_crypto.c b/test/performance/odp_crypto.c
index 5fd339d..6bd5cf5 100644
--- a/test/performance/odp_crypto.c
+++ b/test/performance/odp_crypto.c
@@ -726,6 +726,8 @@  int main(int argc, char *argv[])
 	odp_cpumask_t cpumask;
 	char cpumaskstr[ODP_CPUMASK_STR_SIZE];

+	memset(&cargs, 0, sizeof(cargs));
+
 	/* Parse and store the application arguments */
 	parse_args(argc, argv, &cargs);