OSSP CVS Repository

ossp - Difference in ossp-pkg/xds/xdr-encode-int64.c versions 1.3 and 1.3.2.1
Not logged in
[Honeypot]  [Browse]  [Home]  [Login]  [Reports
[Search]  [Ticket]  [Timeline
  [History

ossp-pkg/xds/xdr-encode-int64.c 1.3 -> 1.3.2.1

--- xdr-encode-int64.c   2001/07/23 15:33:42     1.3
+++ xdr-encode-int64.c   2001/08/01 11:26:56     1.3.2.1
@@ -28,41 +28,48 @@
 #include <assert.h>
 #include "internal.h"
 
-int xdr_encode_int64(xds_t* xds, void* engine_context, void* buffer, size_t buffer_size, va_list* args)
+int xdr_encode_int64(xds_t* xds, void* engine_context,
+                     void* buffer, size_t buffer_size, size_t* used_buffer_size,
+                     va_list* args)
     {
+    xds_uint64_t tmp;
+    xds_int64_t  value;
+
     /* Consistency checks. */
 
-    assert(xds != NULL);
-    assert(buffer != NULL);
-    assert(buffer_size != 0);
-    assert(args != NULL);
-    if (xds == NULL || buffer == NULL || buffer_size == 0 || args == NULL)
-        return XDS_ERR_INVALID_ARG;
+    xds_check_parameter(xds != NULL);
+    xds_check_parameter(buffer != NULL);
+    xds_check_parameter(buffer_size != 0);
+    xds_check_parameter(args != NULL);
+
+    /* We need 8 bytes in the buffer to format our parameter. If we
+       don't have them, return an overflow error. */
+
+    if (buffer_size < 8)
+        return XDS_ERR_OVERFLOW;
+    else
+        *used_buffer_size = 8;
 
     /* Get value and format it into the buffer. */
 
-    if (buffer_size >= 8)
+    value = va_arg(*args, xds_uint64_t);
+    if (value < 0)
         {
-        xds_uint64_t tmp;
-        xds_int64_t  value = va_arg(*args, xds_uint64_t);
-        if (value < 0)
-            {
-            value = 0 - value;
-            tmp = 0 - (xds_uint64_t)value;
-            }
-        else
-            tmp = (xds_uint64_t)value;
-        ((xds_uint8_t*)buffer)[0] = (tmp >> 56) & 0x000000ff;
-        ((xds_uint8_t*)buffer)[1] = (tmp >> 48) & 0x000000ff;
-        ((xds_uint8_t*)buffer)[2] = (tmp >> 40) & 0x000000ff;
-        ((xds_uint8_t*)buffer)[3] = (tmp >> 32) & 0x000000ff;
-        ((xds_uint8_t*)buffer)[4] = (tmp >> 24) & 0x000000ff;
-        ((xds_uint8_t*)buffer)[5] = (tmp >> 16) & 0x000000ff;
-        ((xds_uint8_t*)buffer)[6] = (tmp >>  8) & 0x000000ff;
-        ((xds_uint8_t*)buffer)[7] = (tmp >>  0) & 0x000000ff;
+        value = 0 - value;
+        tmp = 0 - (xds_uint64_t)value;
         }
+    else
+        tmp = (xds_uint64_t)value;
+    ((xds_uint8_t*)buffer)[0] = (tmp >> 56) & 0x000000ff;
+    ((xds_uint8_t*)buffer)[1] = (tmp >> 48) & 0x000000ff;
+    ((xds_uint8_t*)buffer)[2] = (tmp >> 40) & 0x000000ff;
+    ((xds_uint8_t*)buffer)[3] = (tmp >> 32) & 0x000000ff;
+    ((xds_uint8_t*)buffer)[4] = (tmp >> 24) & 0x000000ff;
+    ((xds_uint8_t*)buffer)[5] = (tmp >> 16) & 0x000000ff;
+    ((xds_uint8_t*)buffer)[6] = (tmp >>  8) & 0x000000ff;
+    ((xds_uint8_t*)buffer)[7] = (tmp >>  0) & 0x000000ff;
 
     /* Done. */
 
-    return 8;
+    return XDS_OK;
     }

CVSTrac 2.0.1