Skip to content

Fix UBSan gcc warning for first arg to memset non null#767

Merged
julienmalik merged 1 commit intouclouvain:masterfrom
julienmalik:fix_memset_null_pointer
May 2, 2016
Merged

Fix UBSan gcc warning for first arg to memset non null#767
julienmalik merged 1 commit intouclouvain:masterfrom
julienmalik:fix_memset_null_pointer

Conversation

@julienmalik
Copy link
Copy Markdown
Collaborator

No description provided.

@julienmalik
Copy link
Copy Markdown
Collaborator Author

I manually checked that the NULL case was when w=1 and h=0, so we were actually memsetting zero bytes.

Maybe this should be detected earlier ?

@julienmalik
Copy link
Copy Markdown
Collaborator Author

Updates #718

@julienmalik julienmalik merged commit ba0cf12 into uclouvain:master May 2, 2016
@julienmalik
Copy link
Copy Markdown
Collaborator Author

Only 5 UB left on gcc 5.2 : http://my.cdash.org/viewDynamicAnalysis.php?buildid=953384

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