jepler/fruitjam-doom
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

net: explicitly declare a bit-shifted 1 literal as unsigned

Browse source 
This fixes a future "Shifting signed 32-bit value by 31 bits is
undefined behaviour" cppcheck error.
This commit is contained in:
Fabian Greffrath 2019-01-28 09:25:31 +01:00
parent 79b2fed04c
commit 6478a56726
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/net_packet.c
View file

@ -158,10 +158,10 @@ boolean NET_ReadSInt32(net_packet_t *packet, signed int *data)
{ {
if (NET_ReadInt32(packet, (unsigned int *) data)) if (NET_ReadInt32(packet, (unsigned int *) data))
{ {
if (*data & (1 << 31)) if (*data & (1U << 31))
{ {
*data &= ~(1 << 31); *data &= ~(1U << 31);
*data -= (1 << 31); *data -= (1U << 31);
} }
return true; return true;
} }