Skip to content

Conversation

@sjaeckel
Copy link
Member

@sjaeckel sjaeckel commented Jun 22, 2018

This also fixes #411 and fixes #380

... and re-factors how the strings are built up ...

dupes the fix done in #413 :)

@sjaeckel sjaeckel requested a review from karel-m June 22, 2018 08:04
@karel-m
Copy link
Member

karel-m commented Jun 22, 2018

char number[32], *ptr; >> warning: unused variable ‘number’ [-Wunused-variable]

@sjaeckel
Copy link
Member Author

char number[32], *ptr; >> warning: unused variable ‘number’ [-Wunused-variable]

yeah and some other sloppy mistakes also already fixed...

@karel-m
Copy link
Member

karel-m commented Jun 22, 2018

Looks good - BTW: it also fixes #380

@sjaeckel
Copy link
Member Author

Looks good - BTW: it also fixes #380

that's why I didn't only fix the bug ;)

@sjaeckel sjaeckel merged commit 8972027 into develop Jun 22, 2018
@sjaeckel sjaeckel deleted the fix/411 branch June 22, 2018 09:33
sjaeckel added a commit that referenced this pull request Jun 22, 2018
Fix/411
(cherry picked from commit 8972027)
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.

core dump clang-tidy: clang-analyzer-security.insecureAPI.strcpy

3 participants