Skip to content

fix #13854: memleak might show misleading location#7796

Merged
danmar merged 2 commits intodanmar:mainfrom
ludviggunne:13854
Sep 4, 2025
Merged

fix #13854: memleak might show misleading location#7796
danmar merged 2 commits intodanmar:mainfrom
ludviggunne:13854

Conversation

@ludviggunne
Copy link
Copy Markdown
Collaborator

This sets the column of added braces to the column of the token before it, so we can point to the last token of the scope.

Checking tickets/13854.c ...
tickets/13854.c:3:30: error: Memory leak: p1 [memleak]
        void * p1 = malloc(5);
                             ^

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Sep 4, 2025

@danmar danmar merged commit a674741 into danmar:main Sep 4, 2025
63 checks passed
@ludviggunne ludviggunne deleted the 13854 branch September 4, 2025 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants