-
tsoomeg'morning:)
-
tsoome.oO already found real bug with warning suppression lifted...
-
gitomat[illumos-gate] 15116 find_elf does not find non-executable .so files -- Andy Fiddaman <illumos⊙fn>
-
andyftsoome - nice :)
-
tsoomeerror: '&' within '^'
-
tsoome!(ntohl(*(uint32_t *)buff) & 2^31)
-
tsoomeanother one
-
paulfoh dear wishful wysiwig C
-
tsoomeI'm not really even sure what is the idea there
-
paulflooks to me that someone thought that 2^31 means two to the power of 31
-
paulfmeaning 1 << 31
-
tsoomeyep
-
tsoomeusing xor does not really make much sense
-
jlevontsoome: gcc.gnu.org/bugzilla/show_bug.cgi?id=90885 :P
-
andyfThat looks handy
-
jbkyeah
-
tsoomeheh:)
-
tsoomeanyhow, this is the function and I have no idea why it has this check. src.illumos.org/source/xref/illumos….c?r=48bbca81&mo=44220&fi=1679#1679
-
andyfMe neither..
-
andyfThere is this..
-
jbkif I had to guess.. assuming buff is a complete message, and assuming the first 4 bytes are a network byte order length of the message
-
andyf
-
jbkand assuming it's treating it as a signed value...
-
jbk(which is a lot of assumptions...)
-
tsoomeoh, I see
-
andyfif (!LAST_FRAG()) ?
-
tsoomeyea, that would be helpful for poor human reading this code:D
-
andyf44:#define MAX_XID (2^16)
-
andyfat least there aren't too many in the gate
-
jbkyeah.. that would make the intent clearer