Details
- Reviewers
imp - Commits
- rS329724: Build getty(8) with WARNS=6.
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 14774 Build 14893: arc lint + arc unit
Event Timeline
| libexec/getty/chat.c | ||
|---|---|---|
| 273 | you convert unsigned char * to char * by doing this. | |
| libexec/getty/main.c | ||
| 263 | 32 looks wrong here. It's likely not entirely incorrect, since 32 is > 0, but is a left-over from old-school int days. | |
| 712 | why not make i size_t? | |
| 719 | and that would save the cast here... | |
| libexec/getty/subr.c | ||
| 75 | where does this get freed? | |
| 121 | dba[0] doesn't look to be freed by all the return paths in the switch above. | |
| libexec/getty/chat.c | ||
|---|---|---|
| 273 | Well, the old code did the same. I'm trying to minimize the differences as much as possible, as I have rather limited way of testing this, and I'd rather avoid breaking something due to "cleanup". | |
| libexec/getty/main.c | ||
| 263 | True. Still, I'd like to change it as little as possible. Or perhaps just fix it in a separate commit? | |
So, does it look reasonable enough to get accepted? Thanks!
| libexec/getty/chat.c | ||
|---|---|---|
| 273 | Indeed - but it will be much easier (and safer) to do when we have a proper warnings for those. | |
Should have ticked OK sooner. There's some issues, as noted, but the new bugs introduced have been fixed so they can wait for another day.