Skip to content

Avoiding null pointer dereference#973

Merged
Cyan4973 merged 1 commit intolz4:devfrom
klebertarcisio:patch_fix
Mar 10, 2021
Merged

Avoiding null pointer dereference#973
Cyan4973 merged 1 commit intolz4:devfrom
klebertarcisio:patch_fix

Conversation

@klebertarcisio
Copy link
Copy Markdown

This pull request aims to fix null pointer dereference.

Null pointer dereference

@Cyan4973
Copy link
Copy Markdown
Member

Cyan4973 commented Mar 1, 2021

Yes, that's a good point.

Issue here is C90 compliance (which is the standard used by lz4) : one can't add a statement before a declaration within the same block.

This is a minor issue, there are multiple ways to get this fixed.

@klebertarcisio
Copy link
Copy Markdown
Author

Hi @Cyan4973, thanks for the feedback. I reversed the commit to send another solution considering C90.

@Cyan4973
Copy link
Copy Markdown
Member

Thanks @klebertarcisio !

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