Fix a large pile of solaris warnings for bug 19767.

In nearly all cases, this is a matter of making sure that we include
orconfig.h before we include any standard c headers.

diff --git a/src/common/address.c b/src/common/address.c
index 793a40effc2c6774cc1333bf2aed0a6a65a5866e..c514588d7f15764deb0c74770c427b2937e6002a 100644 (file)
--- a/src/common/address.c
+++ b/src/common/address.c
@@ -1595,6 +1595,7 @@ get_interface_addresses_raw,(int severity, sa_family_t family))
     return result;
 #endif
   (void) severity;
+  (void) result;
   return NULL;
 }
 

diff --git a/src/common/crypto.h b/src/common/crypto.h
index 682c4e3253f70381c7b51caa47b11bdedf13815b..e4d3d06d13c546bcb366952aef2786c21d83a6e9 100644 (file)
--- a/src/common/crypto.h
+++ b/src/common/crypto.h
@@ -13,6 +13,8 @@
 #ifndef TOR_CRYPTO_H
 #define TOR_CRYPTO_H
 
+#include "orconfig.h"
+
 #include <stdio.h>
 #include "torint.h"
 #include "testsupport.h"

diff --git a/src/common/util.c b/src/common/util.c
index f3effe0957b1bd384ca05dcbd06816f54b570af4..a100f375c36696b021325d3dc6a0014b0500b246 100644 (file)
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -4548,13 +4548,13 @@ tor_get_exit_code(process_handle_t *process_handle,
     return PROCESS_EXIT_RUNNING;
   } else if (retval != process_handle->pid) {
     log_warn(LD_GENERAL, "waitpid() failed for PID %d: %s",
-             process_handle->pid, strerror(errno));
+             (int)process_handle->pid, strerror(errno));
     return PROCESS_EXIT_ERROR;
   }
 
   if (!WIFEXITED(stat_loc)) {
     log_warn(LD_GENERAL, "Process %d did not exit normally",
-             process_handle->pid);
+             (int)process_handle->pid);
     return PROCESS_EXIT_ERROR;
   }
 

diff --git a/src/ext/ed25519/donna/ed25519_tor.c b/src/ext/ed25519/donna/ed25519_tor.c
index 52b259dfe10826b05c5d7824bfad7082612d8728..5f3fdc5dae36ebcafc3510ab937f8255494678e9 100644 (file)
--- a/src/ext/ed25519/donna/ed25519_tor.c
+++ b/src/ext/ed25519/donna/ed25519_tor.c
@@ -34,7 +34,7 @@
 #define ED25519_FN2(fn,suffix) ED25519_FN3(fn,suffix)
 #define ED25519_FN(fn)         ED25519_FN2(fn,ED25519_SUFFIX)
 
-
+#include "orconfig.h"
 #include "ed25519-donna.h"
 #include "ed25519_donna_tor.h"
 #include "ed25519-randombytes.h"

diff --git a/src/ext/ed25519/ref10/keypair.c b/src/ext/ed25519/ref10/keypair.c
index 7ddbaa971e7c998545c021730bab11ae23abff44..68a88f9adce34b4a2cd6b7438bf2aef3dc07f9bc 100644 (file)
--- a/src/ext/ed25519/ref10/keypair.c
+++ b/src/ext/ed25519/ref10/keypair.c
@@ -1,6 +1,7 @@
 /* Modified for Tor: new API, 64-byte secret keys. */
-#include <string.h>
+
 #include "randombytes.h"
+#include <string.h>
 #include "crypto_sign.h"
 #include "crypto_hash_sha512.h"
 #include "ge.h"

diff --git a/src/ext/ed25519/ref10/open.c b/src/ext/ed25519/ref10/open.c
index 9dbeb4cdd092d9b294dc19127e09b7fae1538009..3ab7b7d6e71b9ba5e95e6c9416f1c9d0777789ae 100644 (file)
--- a/src/ext/ed25519/ref10/open.c
+++ b/src/ext/ed25519/ref10/open.c
@@ -1,6 +1,7 @@
 /* (Modified by Tor to verify signature separately from message) */
-#include <string.h>
+
 #include "crypto_sign.h"
+#include <string.h>
 #include "crypto_hash_sha512.h"
 #include "crypto_verify_32.h"
 #include "ge.h"

diff --git a/src/or/circuitmux_ewma.c b/src/or/circuitmux_ewma.c
index b784a140ac0e616af7a7564c79ff03dfe2636b79..13836cdcfaed07663b789b61ed108c57b6b7c337 100644 (file)
--- a/src/or/circuitmux_ewma.c
+++ b/src/or/circuitmux_ewma.c
@@ -8,6 +8,8 @@
 
 #define TOR_CIRCUITMUX_EWMA_C_
 
+#include "orconfig.h"
+
 #include <math.h>
 
 #include "or.h"

diff --git a/src/test/test-child.c b/src/test/test-child.c
index e2552a499dbe9bb883affbc052e07c7509ba91df..fdf3ccec0ab41e9887dff6b5303b2dc981e9335f 100644 (file)
--- a/src/test/test-child.c
+++ b/src/test/test-child.c
@@ -1,8 +1,8 @@
 /* Copyright (c) 2011-2016, The Tor Project, Inc. */
 /* See LICENSE for licensing information */
 
-#include <stdio.h>
 #include "orconfig.h"
+#include <stdio.h>
 #ifdef _WIN32
 #define WINDOWS_LEAN_AND_MEAN
 #include <windows.h>

diff --git a/src/test/test-memwipe.c b/src/test/test-memwipe.c
index 5d4fcec664ca223d77c6cc5b400011f12191a0d4..2689b953667e8d8f23626af122a399bc7014d4f9 100644 (file)
--- a/src/test/test-memwipe.c
+++ b/src/test/test-memwipe.c
@@ -1,3 +1,4 @@
+#include "orconfig.h"
 #include <string.h>
 #include <stdio.h>
 #include <sys/types.h>

diff --git a/src/test/test_channeltls.c b/src/test/test_channeltls.c
index 04ae9a6da753ad9bfcf3bd8aa0cf34f67372c059..fda466fc82aaddf3e1e194023f3dafac0455769b 100644 (file)
--- a/src/test/test_channeltls.c
+++ b/src/test/test_channeltls.c
@@ -1,6 +1,8 @@
 /* Copyright (c) 2014-2016, The Tor Project, Inc. */
 /* See LICENSE for licensing information */
 
+#include "orconfig.h"
+
 #include <math.h>
 
 #define TOR_CHANNEL_INTERNAL_

diff --git a/src/test/test_scheduler.c b/src/test/test_scheduler.c
index 6e9889b48baa617610c712ec2e31002c7a9fae23..db331cb662c03459644261d5ac9edff5f8818279 100644 (file)
--- a/src/test/test_scheduler.c
+++ b/src/test/test_scheduler.c
@@ -1,10 +1,10 @@
 /* Copyright (c) 2014-2016, The Tor Project, Inc. */
 /* See LICENSE for licensing information */
 
-#include <math.h>
-
 #include "orconfig.h"
 
+#include <math.h>
+
 /* Libevent stuff */
 #ifdef HAVE_EVENT2_EVENT_H
 #include <event2/event.h>

diff --git a/src/test/test_status.c b/src/test/test_status.c
index 84a0f6c0241be7c6ecc8117c0e35df2bbd494c22..1f4ec0a83efcd3ddbf482ab0b00d896b0cc30fea 100644 (file)
--- a/src/test/test_status.c
+++ b/src/test/test_status.c
@@ -3,6 +3,8 @@
 #define LOG_PRIVATE
 #define REPHIST_PRIVATE
 
+#include "orconfig.h"
+
 #include <float.h>
 #include <math.h>