Make endianness-conversion macros less fragile.

bbf85aaf · Matthieu Boutier · Juliusz Chroboczek · 2b757086 · bbf85aaf
Commit bbf85aaf authored Feb 11, 2012 by Matthieu Boutier Committed by Juliusz Chroboczek Feb 11, 2012
Show whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

util.h util.h +4 -4

No files found.
--- a/util.h
+++ b/util.h
@@ -21,10 +21,10 @@ THE SOFTWARE.
 */

 #if defined(i386) || defined(__mc68020__) || defined(__x86_64__)
-#define DO_NTOHS(_d, _s) do { _d = ntohs(*(unsigned short*)(_s)); } while(0)
-#define DO_NTOHL(_d, _s) do { _d = ntohl(*(unsigned*)(_s)); } while(0)
-#define DO_HTONS(_d, _s) do { *(unsigned short*)(_d) = htons(_s); } while(0)
-#define DO_HTONL(_d, _s) do { *(unsigned*)(_d) = htonl(_s); } while(0)
+#define DO_NTOHS(_d, _s) do{ _d = ntohs(*(const unsigned short*)(_s)); }while(0)
+#define DO_NTOHL(_d, _s) do{ _d = ntohl(*(const unsigned*)(_s)); } while(0)
+#define DO_HTONS(_d, _s) do{ *(unsigned short*)(_d) = htons(_s); } while(0)
+#define DO_HTONL(_d, _s) do{ *(unsigned*)(_d) = htonl(_s); } while(0)
 /* Some versions of gcc seem to be buggy, and ignore the packed attribute.
   Disable this code until the issue is clarified. */
 /* #elif defined __GNUC__*/