Access the string at len - 1 instead of len. Signed-off-by: Roberto Sassu <roberto.sassu@xxxxxxxxxx> --- arch/s390/lib/string.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/arch/s390/lib/string.c b/arch/s390/lib/string.c index cfcdf76d6a95..162a391788ad 100644 --- a/arch/s390/lib/string.c +++ b/arch/s390/lib/string.c @@ -261,12 +261,12 @@ char *strrchr(const char *s, int c) { size_t len = __strend(s) - s; - if (len) - do { - if (s[len] == (char) c) - return (char *) s + len; - } while (--len > 0); - return NULL; + if (len) + do { + if (s[len - 1] == (char) c) + return (char *) s + len - 1; + } while (--len > 0); + return NULL; } EXPORT_SYMBOL(strrchr); #endif -- 2.32.0