]> git.ipfire.org Git - thirdparty/tor.git/commitdiff
config: use ptrdiff_t for all field-offset members.
authorNick Mathewson <nickm@torproject.org>
Wed, 4 Sep 2019 14:50:09 +0000 (10:50 -0400)
committerNick Mathewson <nickm@torproject.org>
Wed, 4 Sep 2019 14:50:09 +0000 (10:50 -0400)
Previously we used int here, but it is more correct to use
ptrdiff_t.  (This never actually matters for our code in practice,
since the structure we are managing here never exceed INT_MAX in
size.)

src/app/config/confparse.h
src/lib/conf/conftypes.h

index b788e2cd1bb0e537e0dcaa1b9df7ea15f6cd24ad..fc07996aabb0c914d0f48d9ba53ab6b2eebf36f5 100644 (file)
@@ -72,7 +72,7 @@ typedef struct config_format_t {
   const struct_member_t *extra;
   /** The position of a config_suite_t pointer within the toplevel object,
    * or -1 if there is no such pointer. */
-  int config_suite_offset;
+  ptrdiff_t config_suite_offset;
 } config_format_t;
 
 /**
index fabad97d0c3c813aad6d6ae95ea034f9e274659d..abfb53b21ba7352650ad4c08cff04b5b02bd8890 100644 (file)
@@ -33,6 +33,8 @@
 #include "lib/conf/conftesting.h"
 #endif
 
+#include <stddef.h>
+
 /** Enumeration of types which option values can take */
 typedef enum config_type_t {
   CONFIG_TYPE_STRING = 0,   /**< An arbitrary string. */
@@ -89,7 +91,7 @@ typedef struct struct_member_t {
    * Offset of this field within the structure.  Compute this with
    * offsetof(structure, fieldname).
    **/
-  int offset;
+  ptrdiff_t offset;
 } struct_member_t;
 
 /**
@@ -102,7 +104,7 @@ typedef struct struct_member_t {
 typedef struct struct_magic_decl_t {
   const char *typename;
   uint32_t magic_val;
-  int magic_offset;
+  ptrdiff_t magic_offset;
 } struct_magic_decl_t;
 
 /**