Clean up rtrim implementation

If I understand the C standard correctly, you can't actually point
at a position immediately _before_ the start of an object; only at the
position immediately after.

According to J.2 in the standard, in its big list of undefined behavior:

 "The behavior is undefined in the following circumstances:
     ...
   — Addition or subtraction of a pointer into, or just beyond, an
    array object and an integer type produces a result that does not
    point into, or just beyond, the same array object (6.5.6)."

So we've got to fix rtrim to not do that.  Also, make it unit tested,
and give it an evutil_*_ name.

1 files changed, 4 insertions, 0 deletions

diff --git a/util-internal.h b/util-internal.h
index da1fb838..dff3d70b 100644
--- a/util-internal.h
+++ b/util-internal.h
@@ -218,6 +218,10 @@ int EVUTIL_ISUPPER_(char c);
 char EVUTIL_TOUPPER_(char c);
 char EVUTIL_TOLOWER_(char c);
 
+/** Remove all trailing whitespace from the end of a string */
+void evutil_rtrim_(char *);
+
+
 /** Helper macro.  If we know that a given pointer points to a field in a
     structure, return a pointer to the structure itself.  Used to implement
     our half-baked C OO.  Example: