all: ensure unsigned numbers are used in shifts
authorFerry Huberts <ferry.huberts@pelagic.nl>
Sat, 30 Apr 2016 20:13:43 +0000 (22:13 +0200)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Sat, 30 Apr 2016 20:13:43 +0000 (22:13 +0200)
Where relevant.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
17 files changed:
lib/bmf/src/NetworkInterfaces.c
lib/bmf/src/PacketHistory.c
lib/bmf/src/PacketHistory.h
lib/p2pd/src/PacketHistory.c
lib/p2pd/src/PacketHistory.h
lib/pud/nmealib/include/nmea/info.h
lib/pud/nmealib/include/nmea/sentence.h
lib/pud/wireformat/src/wireFormat.c
lib/secure/src/olsrd_secure.c
src/cfgparser/olsrd_conf.c
src/duplicate_set.c
src/gateway.h
src/link_set.h
src/mantissa.c
src/scheduler.c
src/scheduler.h
src/tc_set.c

index f1c226c..c76ae77 100644 (file)
@@ -215,14 +215,14 @@ int SetBmfInterfaceIp(
   EtherTunTapIpMask = 0;
   for (i = 0; i < prefixLen; i++)
   {
-    EtherTunTapIpMask |= (1 << (IPV4_MAX_PREFIXLEN - 1 - i));
+    EtherTunTapIpMask |= (1u << (IPV4_MAX_PREFIXLEN - 1 - i));
   }
 
   /* Compose IP broadcast address in host byte order */
   EtherTunTapIpBroadcast = EtherTunTapIp;
   for (i = prefixLen; i < IPV4_MAX_PREFIXLEN; i++)
   {
-    EtherTunTapIpBroadcast |= (1 << (IPV4_MAX_PREFIXLEN - 1 - i));
+    EtherTunTapIpBroadcast |= (1u << (IPV4_MAX_PREFIXLEN - 1 - i));
   }
 
   TunTapIpOverruled = 1;
index 24c605b..dbcfaeb 100644 (file)
@@ -182,7 +182,7 @@ u_int32_t PacketCrc32(unsigned char* ipPacket, ssize_t len)
  * ------------------------------------------------------------------------- */
 u_int32_t Hash(u_int32_t from32)
 {
-  return ((from32 >> N_HASH_BITS) + from32) & ((1 << N_HASH_BITS) - 1);
+  return ((from32 >> N_HASH_BITS) + from32) & ((1u << N_HASH_BITS) - 1);
 } /* Hash */
 
 /* -------------------------------------------------------------------------
@@ -195,7 +195,7 @@ u_int32_t Hash(u_int32_t from32)
  * ------------------------------------------------------------------------- */
 void InitPacketHistory(void)
 {
-  int i;
+  unsigned int i;
 
   GenerateCrc32Table();
 
index 205aff3..0e53515 100644 (file)
@@ -59,7 +59,7 @@
 #include <sys/times.h> /* clock_t */
 
 #define N_HASH_BITS 12
-#define HISTORY_HASH_SIZE (1 << N_HASH_BITS)
+#define HISTORY_HASH_SIZE (1u << N_HASH_BITS)
 
 /* Time-out of duplicate entries, in milliseconds */
 #define HISTORY_HOLD_TIME 3000
index 7ba9221..48b2324 100644 (file)
@@ -182,7 +182,7 @@ u_int32_t PacketCrc32(unsigned char* ipPacket, ssize_t len)
  * ------------------------------------------------------------------------- */
 u_int32_t Hash(u_int32_t from32)
 {
-  return ((from32 >> N_HASH_BITS) + from32) & ((1 << N_HASH_BITS) - 1);
+  return ((from32 >> N_HASH_BITS) + from32) & ((1u << N_HASH_BITS) - 1);
 } /* Hash */
 
 /* -------------------------------------------------------------------------
@@ -195,7 +195,7 @@ u_int32_t Hash(u_int32_t from32)
  * ------------------------------------------------------------------------- */
 void InitPacketHistory(void)
 {
-  int i;
+  unsigned int i;
 
   GenerateCrc32Table();
 
index d68d40b..79bc9b5 100644 (file)
@@ -59,7 +59,7 @@
 #include <sys/times.h> /* clock_t */
 
 #define N_HASH_BITS 15
-#define HISTORY_HASH_SIZE (1 << N_HASH_BITS)
+#define HISTORY_HASH_SIZE (1u << N_HASH_BITS)
 
 /* Time-out of duplicate entries, in milliseconds */
 #define HISTORY_HOLD_TIME 3000
index 398e197..024b3f6 100644 (file)
@@ -173,25 +173,25 @@ typedef struct _nmeaINFO {
  * The values are used in the 'present' mask.
  */
 typedef enum _nmeaINFO_FIELD {
-       SMASK                   = (1 << 0),  /* 0x00001 */
-       UTCDATE                 = (1 << 1),  /* 0x00002 */
-       UTCTIME                 = (1 << 2),  /* 0x00004 */
-       SIG                             = (1 << 3),  /* 0x00008 */
-       FIX                             = (1 << 4),  /* 0x00010 */
-       PDOP                    = (1 << 5),  /* 0x00020 */
-       HDOP                    = (1 << 6),  /* 0x00040 */
-       VDOP                    = (1 << 7),  /* 0x00080 */
-       LAT                             = (1 << 8),  /* 0x00100 */
-       LON                             = (1 << 9),  /* 0x00200 */
-       ELV                             = (1 << 10), /* 0x00400 */
-       SPEED                   = (1 << 11), /* 0x00800 */
-       TRACK                   = (1 << 12), /* 0x01000 */
-       MTRACK                  = (1 << 13), /* 0x02000 */
-       MAGVAR                  = (1 << 14), /* 0x04000 */
-       SATINUSECOUNT   = (1 << 15), /* 0x08000 */
-       SATINUSE                = (1 << 16), /* 0x10000 */
-       SATINVIEW               = (1 << 17), /* 0x20000 */
-       _nmeaINFO_FIELD_LAST = SATINVIEW
+  SMASK         = (1u << 0),  /* 0x00001 */
+  UTCDATE       = (1u << 1),  /* 0x00002 */
+  UTCTIME       = (1u << 2),  /* 0x00004 */
+  SIG           = (1u << 3),  /* 0x00008 */
+  FIX           = (1u << 4),  /* 0x00010 */
+  PDOP          = (1u << 5),  /* 0x00020 */
+  HDOP          = (1u << 6),  /* 0x00040 */
+  VDOP          = (1u << 7),  /* 0x00080 */
+  LAT           = (1u << 8),  /* 0x00100 */
+  LON           = (1u << 9),  /* 0x00200 */
+  ELV           = (1u << 10), /* 0x00400 */
+  SPEED         = (1u << 11), /* 0x00800 */
+  TRACK         = (1u << 12), /* 0x01000 */
+  MTRACK        = (1u << 13), /* 0x02000 */
+  MAGVAR        = (1u << 14), /* 0x04000 */
+  SATINUSECOUNT = (1u << 15), /* 0x08000 */
+  SATINUSE      = (1u << 16), /* 0x10000 */
+  SATINVIEW     = (1u << 17), /* 0x20000 */
+  _nmeaINFO_FIELD_LAST = SATINVIEW
 } nmeaINFO_FIELD;
 
 #define NMEA_INFO_PRESENT_MASK ((_nmeaINFO_FIELD_LAST << 1) - 1)
index dbfd133..38f1c9c 100644 (file)
@@ -36,11 +36,11 @@ extern "C" {
  */
 enum nmeaPACKTYPE {
        GPNON = 0,                      /**< Unknown packet type. */
-       GPGGA = (1 << 0),       /**< GGA - Essential fix data which provide 3D location and accuracy data. */
-       GPGSA = (1 << 1),       /**< GSA - GPS receiver operating mode, SVs used for navigation, and DOP values. */
-       GPGSV = (1 << 2),       /**< GSV - Number of SVs in view, PRN numbers, elevation, azimuth & SNR values. */
-       GPRMC = (1 << 3),       /**< RMC - Recommended Minimum Specific GPS/TRANSIT Data. */
-       GPVTG = (1 << 4)        /**< VTG - Actual track made good and speed over ground. */
+       GPGGA = (1u << 0),      /**< GGA - Essential fix data which provide 3D location and accuracy data. */
+       GPGSA = (1u << 1),      /**< GSA - GPS receiver operating mode, SVs used for navigation, and DOP values. */
+       GPGSV = (1u << 2),      /**< GSV - Number of SVs in view, PRN numbers, elevation, azimuth & SNR values. */
+       GPRMC = (1u << 3),      /**< RMC - Recommended Minimum Specific GPS/TRANSIT Data. */
+       GPVTG = (1u << 4)       /**< VTG - Actual track made good and speed over ground. */
 };
 
 /**
index 227d657..48846ee 100644 (file)
@@ -67,7 +67,7 @@ bool isValidNodeIdType(unsigned long long nodeIdType) {
  */
 
 /** Determine the validity time in seconds from the OLSR wire format value */
-#define PUD_VALIDITY_TIME_FROM_OLSR(msn, lsn) ((((lsn) + 16) * (1 << (msn))) - 16)
+#define PUD_VALIDITY_TIME_FROM_OLSR(msn, lsn) ((((lsn) + 16) * (1u << (msn))) - 16)
 
 /**
  Get the validity time from a message
@@ -106,7 +106,7 @@ void setValidityTime(uint8_t * validityTimeField, unsigned long long validityTim
                lsn = 15;
        } else {
                unsigned long lowerBound = PUD_VALIDITY_TIME_FROM_OLSR(msn, 0);
-               unsigned long resolution = (1 << msn);
+               unsigned long resolution = (1u << msn);
                lsn = ((validityTime - lowerBound + (resolution >> 1)) / resolution);
        }
 
@@ -414,7 +414,7 @@ double getPositionUpdateLatitude(
        /* take half of the rounding error */
        lat += 0.5;
 
-       lat /= (double) (1 << PUD_LATITUDE_BITS);
+       lat /= (double) (1u << PUD_LATITUDE_BITS);
        /* lat is now in [0, 1> */
 
        lat -= 0.5;
@@ -448,12 +448,12 @@ void setPositionUpdateLatitude(
        lat += 0.5;
        /* lat is now in [0, 1] */
 
-       lat *= (double) (1 << PUD_LATITUDE_BITS);
+       lat *= (double) (1u << PUD_LATITUDE_BITS);
        /* lat is now in [0, LATITUDE_BITS] */
 
        /* clip max */
-       if (unlikely(lat > (double)((1 << PUD_LATITUDE_BITS) - 1))) {
-               lat = (double) ((1 << PUD_LATITUDE_BITS) - 1);
+       if (unlikely(lat > (double)((1u << PUD_LATITUDE_BITS) - 1))) {
+               lat = (double) ((1u << PUD_LATITUDE_BITS) - 1);
        }
        /* lat is now in [0, 2^LATITUDE_BITS> */
 
@@ -479,7 +479,7 @@ double getPositionUpdateLongitude(
        /* take half of the rounding error */
        lon += 0.5;
 
-       lon /= (1 << PUD_LONGITUDE_BITS);
+       lon /= (1u << PUD_LONGITUDE_BITS);
        /* lon is now in [0, 1> */
 
        lon -= 0.5;
@@ -513,12 +513,12 @@ void setPositionUpdateLongitude(
        lon += 0.5;
        /* lon is now in [0, 1] */
 
-       lon *= (double) (1 << PUD_LONGITUDE_BITS);
+       lon *= (double) (1u << PUD_LONGITUDE_BITS);
        /* lon is now in [0, LONGITUDE_BITS] */
 
        /* clip max */
-       if (unlikely(lon > (double)((1 << PUD_LATITUDE_BITS) - 1))) {
-               lon = (double) ((1 << PUD_LATITUDE_BITS) - 1);
+       if (unlikely(lon > (double)((1u << PUD_LATITUDE_BITS) - 1))) {
+               lon = (double) ((1u << PUD_LATITUDE_BITS) - 1);
        }
 
        /* lon is now in [0, 2^LONGITUDE_BITS> */
index 9aa047d..de78f9f 100644 (file)
@@ -546,7 +546,7 @@ send_challenge(struct interface_olsr *olsr_if, const union olsr_ip_addr *new_hos
 
   /* Set the size including OLSR packet size */
 
-  challenge = olsr_random() << 16;
+  challenge = (uint32_t)olsr_random() << 16;
   challenge |= olsr_random();
 
   /* initialise rrmsg */
index 04aa36a..9b0312a 100644 (file)
@@ -796,7 +796,7 @@ olsrd_sanity_check_cnf(struct olsrd_config *cnf)
 
       uint32_t tablesLow;
       uint32_t tablesHigh;
-      uint32_t tablesLowMax = ((1 << 31) - nrOfTables + 1);
+      uint32_t tablesLowMax = ((1u << 31) - nrOfTables + 1);
 
       uint32_t rulesLow;
       uint32_t rulesHigh;
index 7449019..a2111d5 100644 (file)
@@ -176,7 +176,7 @@ olsr_message_is_duplicate(union olsr_message *m)
 
   entry->too_low_counter = 0;
   if (diff <= 0) {
-    uint32_t bitmask = 1 << ((uint32_t) (-diff));
+    uint32_t bitmask = 1u << ((uint32_t) (-diff));
 
     if ((entry->array & bitmask) != 0) {
       OLSR_PRINTF(9, "blocked 0x%x (diff=%d,mask=%08x) from %s\n", seqnr, diff, entry->array, olsr_ip_to_string(&buf, mainIp));
index 1e73f3b..6d0ee08 100644 (file)
@@ -77,11 +77,11 @@ static inline bool multi_gateway_mode(void) {
 
 /** gateway HNA flags */
 enum gateway_hna_flags {
-  GW_HNA_FLAG_LINKSPEED   = 1 << 0,
-  GW_HNA_FLAG_IPV4        = 1 << 1,
-  GW_HNA_FLAG_IPV4_NAT    = 1 << 2,
-  GW_HNA_FLAG_IPV6        = 1 << 3,
-  GW_HNA_FLAG_IPV6PREFIX  = 1 << 4
+  GW_HNA_FLAG_LINKSPEED   = 1u << 0,
+  GW_HNA_FLAG_IPV4        = 1u << 1,
+  GW_HNA_FLAG_IPV4_NAT    = 1u << 2,
+  GW_HNA_FLAG_IPV6        = 1u << 3,
+  GW_HNA_FLAG_IPV6PREFIX  = 1u << 4
 };
 
 /** gateway HNA field byte offsets in the netmask field of the HNA */
index e4088d9..4e44c9d 100644 (file)
@@ -57,7 +57,7 @@
 
 #define MID_ALIAS_HACK_VTIME  10.0
 
-#define LINK_LOSS_MULTIPLIER (1<<16)
+#define LINK_LOSS_MULTIPLIER (1u<<16)
 
 struct link_entry {
   union olsr_ip_addr local_iface_addr;
index 5262309..2a621cf 100644 (file)
@@ -119,9 +119,9 @@ reltime_to_me(const olsr_reltime interval)
        */
 
       if (b >= 5) {
-        a = (interval - (125 << (b - 1))) / (125 << (b - 5));
+        a = (interval - (125u << (b - 1))) / (125u << (b - 5));
       } else {
-        a = (interval - (125 << (b - 1))) * (1 << (5 - b)) / 125;
+        a = (interval - (125u << (b - 1))) * (1u << (5 - b)) / 125;
       }
 
       b += a >> 4;
index bb73c0c..ec42995 100644 (file)
@@ -602,7 +602,7 @@ calc_jitter(unsigned int rel_time, uint8_t jitter_pct, unsigned int random_val)
    * No jitter or, jitter larger than 99% does not make sense.
    * Also protect against overflows resulting from > 25 bit timers.
    */
-  if (jitter_pct == 0 || jitter_pct > 99 || rel_time > (1 << 24)) {
+  if (jitter_pct == 0 || jitter_pct > 99 || rel_time > (1u << 24)) {
     return GET_TIMESTAMP(rel_time);
   }
 
index e39723a..e99d450 100644 (file)
@@ -93,8 +93,8 @@ LISTNODE2STRUCT(list2timer, struct timer_entry, timer_list);
 #define OLSR_TIMER_PERIODIC   1 /* Periodic timer */
 
 /* Timer flags */
-#define OLSR_TIMER_RUNNING  ( 1 << 0)   /* this timer is running */
-#define OLSR_TIMER_REMOVED  ( 1 << 1)   /* this timer is tagged for removal */
+#define OLSR_TIMER_RUNNING  ( 1u << 0)   /* this timer is running */
+#define OLSR_TIMER_REMOVED  ( 1u << 1)   /* this timer is tagged for removal */
 
 /* Timers */
 void olsr_init_timers(void);
index 27ce00f..c1d250c 100644 (file)
@@ -741,8 +741,8 @@ olsr_calculate_tc_border(uint8_t lower_border, union olsr_ip_addr *lower_border_
     for (i = 0; i < lower_border / 8; i++) {
       lower_border_ip->v6.s6_addr[olsr_cnf->ipsize - i - 1] = 0;
     }
-    lower_border_ip->v6.s6_addr[olsr_cnf->ipsize - lower_border / 8 - 1] &= (0xff << (lower_border & 7));
-    lower_border_ip->v6.s6_addr[olsr_cnf->ipsize - lower_border / 8 - 1] |= (1 << (lower_border & 7));
+    lower_border_ip->v6.s6_addr[olsr_cnf->ipsize - lower_border / 8 - 1] &= (0xffu << (lower_border & 7));
+    lower_border_ip->v6.s6_addr[olsr_cnf->ipsize - lower_border / 8 - 1] |= (1u << (lower_border & 7));
   }
 
   if (upper_border == 0xff) {
@@ -755,8 +755,8 @@ olsr_calculate_tc_border(uint8_t lower_border, union olsr_ip_addr *lower_border_
     for (i = 0; i < upper_border / 8; i++) {
       upper_border_ip->v6.s6_addr[olsr_cnf->ipsize - i - 1] = 0;
     }
-    upper_border_ip->v6.s6_addr[olsr_cnf->ipsize - upper_border / 8 - 1] &= (0xff << (upper_border & 7));
-    upper_border_ip->v6.s6_addr[olsr_cnf->ipsize - upper_border / 8 - 1] |= (1 << (upper_border & 7));
+    upper_border_ip->v6.s6_addr[olsr_cnf->ipsize - upper_border / 8 - 1] &= (0xffu << (upper_border & 7));
+    upper_border_ip->v6.s6_addr[olsr_cnf->ipsize - upper_border / 8 - 1] |= (1u << (upper_border & 7));
   }
   return 1;
 }