[CDRIVER-2308] Run live tests through valgrind Created: 29/Sep/17  Updated: 28/Oct/23  Resolved: 08/Nov/17

Status: Closed
Project: C Driver
Component/s: libmongoc, tests
Affects Version/s: None
Fix Version/s: 1.9.0

Type: Improvement Priority: Major - P3
Reporter: Kevin Albertson Assignee: Kevin Albertson
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Related
is related to CDRIVER-2354 Investigate valgrind warnings about b... Closed

 Description   

Currently the debug-compile-valgrind task on evergreen skips all live tests.
example We should enable this on live tests including replica sets and sharded clusters, so we can get more leak detection coverage.



 Comments   
Comment by A. Jesse Jiryu Davis [ 08/Nov/17 ]

I'm marking this resolved, and putting off until 1.10 an investigation of the printf "jump based on uninitialized" warnings.

Comment by Githook User [ 25/Oct/17 ]

Author:

{'email': 'jesse@mongodb.com', 'name': 'A. Jesse Jiryu Davis', 'username': 'ajdavis'}

Message: Merge remote-tracking branch 'upstream/master'

Comment by Kevin Albertson [ 24/Oct/17 ]

We should investigate the conditional jump errors that are currently being suppressed here: https://github.com/mongodb/mongo-c-driver/blob/f9f6338ed4f6effa7bfb01f93827195ba59b40e3/valgrind.suppressions#L56-L71

They might be unavoidable and should be suppressed, but I didn't take a deep look.

Comment by A. Jesse Jiryu Davis [ 24/Oct/17 ]

Kevin what works remains on this ticket?

Comment by Githook User [ 18/Oct/17 ]

Author:

{'email': 'kevin.albertson@10gen.com', 'name': 'Kevin Albertson', 'username': 'kevinAlbs'}

Message: CDRIVER-2308 run live tests through valgrind
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/f9f6338ed4f6effa7bfb01f93827195ba59b40e3

Comment by Kevin Albertson [ 13/Oct/17 ]

A couple notes:

  • valgrind detects the test /Collection/update/oversize as having a leak. I suspect this is because it is allocating a very large (20mb) char*. But it does seem to free/destroy properly.
  • some tests were producing "conditional jump uninitialized value" errors, which are currently being suppressed until further investigation
Comment by Githook User [ 13/Oct/17 ]

Author:

{'email': 'kevin.albertson@10gen.com', 'name': 'Kevin Albertson', 'username': 'kevinAlbs'}

Message: CDRIVER-2308 fix leaks found from live tests
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/89c7ea10723e1fc1dc0405a26e4997a1df7af226

Generated at Wed Feb 07 21:14:50 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.