Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

lib: kdb.c memory leak #4925

Closed
hannes99 opened this issue May 14, 2023 · 0 comments
Closed

lib: kdb.c memory leak #4925

hannes99 opened this issue May 14, 2023 · 0 comments
Assignees
@hannes99

Comments

@hannes99
Copy link
Contributor

Steps to Reproduce the Problem

kdb meta set spec:/tests/set mountpoint /tmp/spec.ni
kdb meta set spec:/tests/set/some/testKey1 type unsigned_short
kdb spec-mount /tests/set
kdb set -f user:/tests/set/some/testKey1 -5

# causes it
kdb validate user:/tests/set/some

Looks like the variables are not freed properly in case of an error:

https://github.com/ElektraInitiative/libelektra/blob/d8a5905ad3073934d0dc56f508bed0bb6012b996/src/libs/elektra/kdb.c#L1928-L1946C3?plain=1

Expected Result

no memory leak

Actual Result

memory leak

System Information

  • Elektra: master
  • Operating System: Arch Linux

valgrind-3.21.0

Further Log Files and Output
@hannes99 hannes99 self-assigned this May 14, 2023
@tmakar tmakar mentioned this issue May 15, 2023
Merged
19 tasks
hannes99 added a commit that referenced this issue May 15, 2023
@hannes99
lib: memory leak, fixes #4925
d4f3e6e
hannes99 added a commit that referenced this issue May 15, 2023
@hannes99
lib: memory leak, fixes #4925
f2f54f6
@hannes99 hannes99 mentioned this issue May 15, 2023
Merged
19 tasks
atmaxinger added a commit that referenced this issue May 15, 2023
@atmaxinger
Merge pull request #4926 from ElektraInitiative/memleak-4922
af273eb 
lib: memory leak, fixes #4925
flo91 pushed a commit to flo91/libelektra that referenced this issue Jul 12, 2023
@hannes99 @flo91
lib: memory leak, fixes ElektraInitiative#4925
d621da0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@hannes99 hannes99

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hannes99