Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Match malloc signature to its use #674

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

r-barnes
Copy link

@r-barnes r-barnes commented Oct 2, 2024

No description provided.

@Mightyjo
Copy link
Contributor

Hi @r-barnes! That code isn't used by very many folks. What system are you using it on?

@westes
Copy link
Owner

westes commented Oct 23, 2024

@r-barnes! can you explain what problem you encountered that prompted you to submit this pr?

@r-barnes
Copy link
Author

@Mightyjo @westes - I'm using Cent OS9 with the LLVM-15 compiler this was identified by one of the warning flags I have enabled. Regretfully, I don't now recall which one.

@Mightyjo
Copy link
Contributor

That's fair. Weird that it was being built on that machine, though. It should have been flagged out during configure since you have a working malloc.

@westes
Copy link
Owner

westes commented Nov 12, 2024

@r-barnes please let us know what compiler flags triggered this for you. It may be relevant to what is going on. Is this triggered with any scanner or did you need to do something specific to get this to happen? If so, can you post the smallest example scanner so we can understand what the problem is?

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.

3 participants