# HG changeset patch # User Matti Hamalainen # Date 1528217828 -10800 # Node ID 311b14855a1e2d4ee611e3af1af91d0f01e29553 # Parent bfba31c09f0b994f0e0b306c7013f3ec86e25e75 Ehm .. the boolean logic conditions used in dmGrowBufPutU{16,32}{LE,BE}() were reversed due to brainfart. Fixed. diff -r bfba31c09f0b -r 311b14855a1e src/dmgrowbuf.c --- a/src/dmgrowbuf.c Tue Jun 05 19:50:16 2018 +0300 +++ b/src/dmgrowbuf.c Tue Jun 05 19:57:08 2018 +0300 @@ -288,14 +288,14 @@ if (buf->literal && buf->backwards) { return - !dmGrowBufPutU8(buf, val & 0xff) || - !dmGrowBufPutU8(buf, (val >> 8) & 0xff); + dmGrowBufPutU8(buf, val & 0xff) && + dmGrowBufPutU8(buf, (val >> 8) & 0xff); } else { return - !dmGrowBufPutU8(buf, (val >> 8) & 0xff) || - !dmGrowBufPutU8(buf, val & 0xff); + dmGrowBufPutU8(buf, (val >> 8) & 0xff) && + dmGrowBufPutU8(buf, val & 0xff); } } @@ -305,14 +305,14 @@ if (buf->literal && buf->backwards) { return - !dmGrowBufPutU8(buf, (val >> 8) & 0xff) || - !dmGrowBufPutU8(buf, val & 0xff); + dmGrowBufPutU8(buf, (val >> 8) & 0xff) && + dmGrowBufPutU8(buf, val & 0xff); } else { return - !dmGrowBufPutU8(buf, val & 0xff) || - !dmGrowBufPutU8(buf, (val >> 8) & 0xff); + dmGrowBufPutU8(buf, val & 0xff) && + dmGrowBufPutU8(buf, (val >> 8) & 0xff); } } @@ -322,18 +322,18 @@ if (buf->literal && buf->backwards) { return - !dmGrowBufPutU8(buf, (val >> 24) & 0xff) || - !dmGrowBufPutU8(buf, (val >> 16) & 0xff) || - !dmGrowBufPutU8(buf, (val >> 8) & 0xff) || - !dmGrowBufPutU8(buf, val & 0xff); + dmGrowBufPutU8(buf, (val >> 24) & 0xff) && + dmGrowBufPutU8(buf, (val >> 16) & 0xff) && + dmGrowBufPutU8(buf, (val >> 8) & 0xff) && + dmGrowBufPutU8(buf, val & 0xff); } else { return - !dmGrowBufPutU8(buf, val & 0xff) || - !dmGrowBufPutU8(buf, (val >> 8) & 0xff) || - !dmGrowBufPutU8(buf, (val >> 16) & 0xff) || - !dmGrowBufPutU8(buf, (val >> 24) & 0xff); + dmGrowBufPutU8(buf, val & 0xff) && + dmGrowBufPutU8(buf, (val >> 8) & 0xff) && + dmGrowBufPutU8(buf, (val >> 16) & 0xff) && + dmGrowBufPutU8(buf, (val >> 24) & 0xff); } } @@ -343,18 +343,18 @@ if (buf->literal && buf->backwards) { return - !dmGrowBufPutU8(buf, val & 0xff) || - !dmGrowBufPutU8(buf, (val >> 8) & 0xff) || - !dmGrowBufPutU8(buf, (val >> 16) & 0xff) || - !dmGrowBufPutU8(buf, (val >> 24) & 0xff); + dmGrowBufPutU8(buf, val & 0xff) && + dmGrowBufPutU8(buf, (val >> 8) & 0xff) && + dmGrowBufPutU8(buf, (val >> 16) & 0xff) && + dmGrowBufPutU8(buf, (val >> 24) & 0xff); } else { return - !dmGrowBufPutU8(buf, (val >> 24) & 0xff) || - !dmGrowBufPutU8(buf, (val >> 16) & 0xff) || - !dmGrowBufPutU8(buf, (val >> 8) & 0xff) || - !dmGrowBufPutU8(buf, val & 0xff); + dmGrowBufPutU8(buf, (val >> 24) & 0xff) && + dmGrowBufPutU8(buf, (val >> 16) & 0xff) && + dmGrowBufPutU8(buf, (val >> 8) & 0xff) && + dmGrowBufPutU8(buf, val & 0xff); } }