llnl / llnl/CHAI

Zero-length ManagedArrays leak memory

Open
#178 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
C++
Stars
111
Forks
23
Avg merge
2d 7h
Merged PRs (30d)
4

Description

We've seen an issue with CHAI allocating zero-sized arrays. We see them report as memory leaks using gcc memory sanitizer, and also with valgrind.

We can see this on a CPU-only build with the default malloc Umpire allocator.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing zero-sized ManagedArray allocation in a CPU-only build using the default malloc Umpire allocator, then run the case under gcc memory sanitizer or valgrind. Trace the allocation and cleanup path for zero-length arrays; done means both tools report no leak.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.