diff options
| author | Simon Glass <[email protected]> | 2022-04-24 23:30:57 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2022-04-25 10:00:03 -0400 |
| commit | d667a0d8f413d7278f912aa4e671bc56d28b25f2 (patch) | |
| tree | 333f3a7ff758dc6b9a32207fc8061c8f15bcca8a /test | |
| parent | 4e64cae0a060c579078de299c6a118aa6f6d7b5b (diff) | |
lib: Fix a few bugs in trailing_strtoln()
At present this has a minor bug in that it reads the byte before the
start of the string, if it is empty. Also it doesn't handle a
non-numeric prefix which is only one character long.
Fix these bugs with a reworked implementation. Add a test for the second
case. The first one is hard to test.
Signed-off-by: Simon Glass <[email protected]>
Diffstat (limited to 'test')
| -rw-r--r-- | test/str_ut.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/test/str_ut.c b/test/str_ut.c index 9674a59f2a6..058b3594379 100644 --- a/test/str_ut.c +++ b/test/str_ut.c @@ -257,6 +257,8 @@ static int str_trailing(struct unit_test_state *uts) ut_asserteq(123, trailing_strtoln(str1, str1 + 6)); ut_asserteq(-1, trailing_strtoln(str1, str1 + 9)); + ut_asserteq(3, trailing_strtol("a3")); + return 0; } STR_TEST(str_trailing, 0); |