[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-changelog] [xen staging] string: avoid undefined behavior in strrchr()



commit 592b5fcb984487f1929d28e1f4eb49d25c2c719d
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Wed Mar 13 11:23:28 2019 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Wed Mar 13 11:23:28 2019 +0100

    string: avoid undefined behavior in strrchr()
    
    The pre-decrement would not only cause misbehavior when wrapping (benign
    because there shouldn't be any NULL pointers passed in), but may also
    create a pointer pointing outside the object that the passed in pointer
    points to (it won't be de-referenced though).
    
    Take the opportunity and also
    - convert bogus space (partly 7 of them) indentation to Linux style tab
      one,
    - add two blank lines.
    
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Reviewed-by: Juergen Gross <jgross@xxxxxxxx>
    Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
 xen/common/string.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/xen/common/string.c b/xen/common/string.c
index 1e122abca8..b8639dd30d 100644
--- a/xen/common/string.c
+++ b/xen/common/string.c
@@ -174,12 +174,13 @@ char *(strchr)(const char *s, int c)
  */
 char *(strrchr)(const char *s, int c)
 {
-       const char *p = s + strlen(s);
-       do {
-           if (*p == (char)c)
-               return (char *)p;
-       } while (--p >= s);
-       return NULL;
+       const char *p = s + strlen(s);
+
+       for (; *p != (char)c; --p)
+               if (p == s)
+                       return NULL;
+
+       return (char *)p;
 }
 #endif
 
--
generated by git-patchbot for /home/xen/git/xen.git#staging

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.