diff options
author | Simon Glass <sjg@chromium.org> | 2022-04-24 23:30:57 -0600 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2022-04-25 10:00:03 -0400 |
commit | d667a0d8f413d7278f912aa4e671bc56d28b25f2 (patch) | |
tree | 333f3a7ff758dc6b9a32207fc8061c8f15bcca8a /test | |
parent | 4e64cae0a060c579078de299c6a118aa6f6d7b5b (diff) | |
download | u-boot-d667a0d8f413d7278f912aa4e671bc56d28b25f2.zip u-boot-d667a0d8f413d7278f912aa4e671bc56d28b25f2.tar.gz u-boot-d667a0d8f413d7278f912aa4e671bc56d28b25f2.tar.bz2 |
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 <sjg@chromium.org>
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 9674a59..058b359 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); |