Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use str* API in place of isdigit() in server/conf.c #676

Open
clepple opened this issue Mar 13, 2019 · 1 comment · May be fixed by #677
Open

use str* API in place of isdigit() in server/conf.c #676

clepple opened this issue Mar 13, 2019 · 1 comment · May be fixed by #677
Assignees
Labels
C-str Issues and PRs about C/C++ methods, headers and data types dealing with strings and memory blocks refactor/fightwarn PR or issue proposal to improve code maintainability without functional changes, or to fix warnings
Milestone

Comments

@clepple
Copy link
Member

clepple commented Mar 13, 2019

isdigit() takes a char/int, not a char*.

Originally mentioned here, but we need to fix this before the next release.

@clepple clepple added this to the 2.7.5 milestone Mar 13, 2019
@aquette aquette self-assigned this Mar 13, 2019
aquette added a commit to aquette/nut that referenced this issue Mar 13, 2019
@aquette aquette linked a pull request Mar 13, 2019 that will close this issue
@aquette aquette added the ready / code review Author (and CI) consider the PR worthy of human rewievers' time label Mar 13, 2019
jimklimov added a commit to jimklimov/nut that referenced this issue Apr 5, 2019
Solution: as a quick fix, pick the first char of the string we pass.
The proper fix would be to use the str_*() API for safe conversions,
in a later iteration.

See-also: networkupstools#676

Signed-off-by: Jim Klimov <[email protected]>
jimklimov added a commit to jimklimov/nut that referenced this issue Apr 5, 2019
Solution: as a quick fix, pick the first char of the string we pass.
The proper fix would be to use the str_*() API for safe conversions,
in a later iteration.

See-also: networkupstools#676

Signed-off-by: Jim Klimov <[email protected]>
jimklimov added a commit to jimklimov/nut that referenced this issue Apr 5, 2019
Solution: as a quick fix, pick the first char of the string we pass.
The proper fix would be to use the str_*() API for safe conversions,
in a later iteration.

See-also: networkupstools#676

Signed-off-by: Jim Klimov <[email protected]>
aquette pushed a commit that referenced this issue Apr 5, 2019
Solution: as a quick fix, pick the first char of the string we pass.
The proper fix would be to use the str_*() API for safe conversions,
in a later iteration.

See-also: #676

Signed-off-by: Jim Klimov <[email protected]>
@clepple clepple removed the ready / code review Author (and CI) consider the PR worthy of human rewievers' time label Apr 29, 2019
@clepple clepple changed the title fix invocation of isdigit(), or use str* API use str* API in place of isdigit() in server/conf.c Apr 29, 2019
@clepple clepple modified the milestones: 2.7.5, 2.7.6 Apr 29, 2019
@clepple
Copy link
Member Author

clepple commented Apr 29, 2019

I'm bumping the milestone from 2.7.5 to 2.7.6 because we merged #690, which fixes the misuse of isdigit(). I still think that a slightly reworked version of #677 (with signed/unsigned expanded out into separate variables, etc.) should be merged into the following release.

@jimklimov jimklimov added the refactor/fightwarn PR or issue proposal to improve code maintainability without functional changes, or to fix warnings label Nov 14, 2021
@jimklimov jimklimov modified the milestones: 2.8.1, 2.8.2 Jan 6, 2023
@jimklimov jimklimov modified the milestones: 2.8.2, 2.8.3 Apr 4, 2024
@jimklimov jimklimov added the C-str Issues and PRs about C/C++ methods, headers and data types dealing with strings and memory blocks label Apr 10, 2024
@jimklimov jimklimov modified the milestones: 2.8.3, 2.8.4 Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-str Issues and PRs about C/C++ methods, headers and data types dealing with strings and memory blocks refactor/fightwarn PR or issue proposal to improve code maintainability without functional changes, or to fix warnings
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants