Re: [PATCH 5/5] bcrypt/tests: Add BCryptEncrypt/Decrypt tests

[Marvin]

Hi,

While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://testbot.winehq.org/JobDetails.pl?Key=27070

Your paranoid android.


=== wvistau64 (64 bit bcrypt) ===
bcrypt.c:792: Test failed: got 598
bcrypt.c:848: Test failed: got 598

=== w2008s64 (64 bit bcrypt) ===
bcrypt.c:792: Test failed: got 598
bcrypt.c:848: Test failed: got 598

=== w7pro64 (64 bit bcrypt) ===
bcrypt.c:792: Test failed: got 622
bcrypt.c:848: Test failed: got 622

=== w864 (64 bit bcrypt) ===
bcrypt.c:792: Test failed: got 654
bcrypt.c:848: Test failed: got 654

=== w1064 (64 bit bcrypt) ===
bcrypt.c:792: Test failed: got 654
bcrypt.c:848: Test failed: got 654

[PATCH 5/5] bcrypt/tests: Add BCryptEncrypt/Decrypt tests

[Bruno Jesus]

On Fri, Dec 2, 2016 at 4:36 PM, Marvin <testbot(a)winehq.org> wrote:
> Hi,
>
> While running your changed tests on Windows, I think I found new failures.
> Being a bot and all I'm not very good at pattern recognition, so I might be
> wrong, but could you please double-check?
> Full results can be found at
> https://testbot.winehq.org/JobDetails.pl?Key=27070
>
> Your paranoid android.
>
> === wvistau64 (64 bit bcrypt) ===
> bcrypt.c:792: Test failed: got 598
> bcrypt.c:848: Test failed: got 598
>
> === w2008s64 (64 bit bcrypt) ===
> bcrypt.c:792: Test failed: got 598
> bcrypt.c:848: Test failed: got 598
>
> === w7pro64 (64 bit bcrypt) ===
> bcrypt.c:792: Test failed: got 622
> bcrypt.c:848: Test failed: got 622
>
> === w864 (64 bit bcrypt) ===
> bcrypt.c:792: Test failed: got 654
> bcrypt.c:848: Test failed: got 654
>
> === w1064 (64 bit bcrypt) ===
> bcrypt.c:792: Test failed: got 654
> bcrypt.c:848: Test failed: got 654

When I run the tests manually the results are OK, I have no idea why
when testbot ran then automatically everything failed... I guess I'll
have to accept these values too.

[PATCH 5/5] bcrypt/tests: Add BCryptEncrypt/Decrypt tests

[Bruno Jesus]

On Fri, Dec 2, 2016 at 5:08 PM, Bruno Jesus <00cpxxx(a)gmail.com> wrote:
> On Fri, Dec 2, 2016 at 4:36 PM, Marvin <testbot(a)winehq.org> wrote:
> > Hi,
> >
> > While running your changed tests on Windows, I think I found new failures.
> > Being a bot and all I'm not very good at pattern recognition, so I might be
> > wrong, but could you please double-check?
> > Full results can be found at
> > https://testbot.winehq.org/JobDetails.pl?Key=27070
> >
> > Your paranoid android.
> >
> > === wvistau64 (64 bit bcrypt) ===
> > bcrypt.c:792: Test failed: got 598
> > bcrypt.c:848: Test failed: got 598
> >
> > === w2008s64 (64 bit bcrypt) ===
> > bcrypt.c:792: Test failed: got 598
> > bcrypt.c:848: Test failed: got 598
> >
> > === w7pro64 (64 bit bcrypt) ===
> > bcrypt.c:792: Test failed: got 622
> > bcrypt.c:848: Test failed: got 622
> >
> > === w864 (64 bit bcrypt) ===
> > bcrypt.c:792: Test failed: got 654
> > bcrypt.c:848: Test failed: got 654
> >
> > === w1064 (64 bit bcrypt) ===
> > bcrypt.c:792: Test failed: got 654
> > bcrypt.c:848: Test failed: got 654
>
> When I run the tests manually the results are OK, I have no idea why
> when testbot ran then automatically everything failed... I guess I'll
> have to accept these values too.

Replying to myself: The size differs between 32 and 64 bits.