Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

check for XXH_malloc NULL return#1778

Merged
dirk-thomas merged 1 commit intoros:melodic-develfrom
artivis:fix/1753
Aug 7, 2019
Merged

check for XXH_malloc NULL return#1778
dirk-thomas merged 1 commit intoros:melodic-develfrom
artivis:fix/1753

Conversation

@artivis
Copy link
Copy Markdown
Contributor

@artivis artivis commented Aug 1, 2019

Edit: I must be tired as I missed #1765... This PR is therefore nearly a duplicate, only diff is here.

fix #1753

  fix ros#1753

Signed-off-by: artivis <jeremie.deray@canonical.com>
@artivis
Copy link
Copy Markdown
Contributor Author

artivis commented Aug 1, 2019

@danielwangksu FYI

@danielwangksu
Copy link
Copy Markdown
Contributor

@danielwangksu FYI

Cool! Looks like found the same thing. i will close mine

@artivis
Copy link
Copy Markdown
Contributor Author

artivis commented Aug 1, 2019

Cool! Looks like found the same thing. i will close mine

Sorry I didn't mean for you to close your PR ^^.

@cwecht
Copy link
Copy Markdown
Contributor

cwecht commented Aug 3, 2019

LGTM

@dirk-thomas dirk-thomas added the bug label Aug 7, 2019
@dirk-thomas
Copy link
Copy Markdown
Member

Thank you for the fix.

@dirk-thomas dirk-thomas merged commit 106edda into ros:melodic-devel Aug 7, 2019
dirk-thomas pushed a commit that referenced this pull request Aug 4, 2020
fix #1753

Signed-off-by: artivis <jeremie.deray@canonical.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

unimplemented null return - malloc may return NULL

4 participants