test_deflate_comp_decomp function is common for all test cases.
The options for this function are inside struct interim_data_params,
which is passed to test_deflate_comp_decomp function as a pointer.
The field buf_idx should be initialized because is used inside
test_deflate_comp_decomp
That's the reason of:
int_data.buf_idx = &i;
I'm not an author of this solution - sorry.
We can review this and try to add new solution.
Thx, Tomek
> -----Original Message-----
> From: Thomas Monjalon [mailto:[email protected]]
> Sent: Friday, April 19, 2019 1:26 PM
> To: Jozwiak, TomaszX <[email protected]>
> Cc: [email protected]; Richardson, Bruce <[email protected]>;
> [email protected]; Trahe, Fiona <[email protected]>;
> [email protected]; Cel, TomaszX <[email protected]>
> Subject: Re: [dpdk-dev] [PATCH] test/compress: fix max mbuf size test case
>
> 19/04/2019 12:57, Jozwiak, TomaszX:
> > > What is supposed to be "i"?
> > > It is initialized at 0 and never touched.
> > >
> > It's touched inside test_deflate_comp_decomp function.
>
> What do you mean?
> It's a local variable and its address is referenced:
> int_data.buf_idx = &i;
> It looks really wrong.
>
> Another error is seen in FreeBSD.
>
> Should I totally revert this patch?
>
>
