uri: Avoid typecasting confusion on s390x

Like memcpy(), the 'void *' argument of p11_buffer_add() points to the
memory area ordered in host's endianness.  Add typecast of int->char to
avoid the confusion.

Reported by Andreas Metzler in:
https://lists.freedesktop.org/archives/p11-glue/2017-January/000633.html

diff --git a/p11-kit/uri.c b/p11-kit/uri.c
index 75d2e846ad1c9fb9449bfee3b68b52a9f3e66801..2659fab836152037c8274664fab067259244e3d0 100644 (file)
--- a/p11-kit/uri.c
+++ b/p11-kit/uri.c
@@ -765,8 +765,10 @@ format_name_equals (p11_buffer *buffer,
                     enum uri_sep *sep,
                     const char *name)
 {
-       if (*sep)
-               p11_buffer_add (buffer, sep, 1);
+       if (*sep) {
+               char c = *sep;
+               p11_buffer_add (buffer, &c, 1);
+       }
        p11_buffer_add (buffer, name, -1);
        p11_buffer_add (buffer, "=", 1);