[gnutls-devel] GnuTLS | tests: enable all tests to run under valgrind (!1383)

[Read-only notification of GnuTLS library development activities]

Daiki Ueno commented on a discussion: https://gitlab.com/gnutls/gnutls/-/merge_requests/1383#note_521658462

> I don't see why the test is irrelevant to "only open /dev/urandom when gnutls_global_init() is called and not before" of https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=760476 I'd appreciate some clarification here.

I generally agree with you, though the test didn't properly exercise the scenario on some cases, because it assumes that /dev/urandom is always associated with fd 3. I guess one way to reliably do that check would be to (1) replace gnutls_rnd as other tests do and (2) in that replacement function, call stat and compare the inode number against a newly opened /dev/urandom.

> What's the policy of what to call and not call under valgrind in shell script tests? I've picked one test at random (tests/cert-tests/certtool.sh) and I see a mixture of calls to certtool, some do call valgrind, some don't.

That sounds like a bug ;-)

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/-/merge_requests/1383#note_521658462
You're receiving this email because of your account on gitlab.com.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.gnupg.org/pipermail/gnutls-devel/attachments/20210304/22040a9c/attachment.html>