commit f4ad0146e973695127d1c9e1769cd2a7882184d8 from: Omar Polo date: Wed Jan 19 19:43:23 2022 UTC work around wstat limitation the previous commit added an explicit check for extra data in messages. The problem is that wstat voluntarily skips reading some fields at the end, so the len will always be greater than zero! commit - bcdb46a77ed7b4ff4d9e7ca8fae79fec74b504ae commit + f4ad0146e973695127d1c9e1769cd2a7882184d8 blob - bb9d9fac4523bf681bf7556b71c3718ea521e992 blob + c4a49b0b5461ea55a8a4a5010cc6ea4fddc0d9b6 --- kamid/client.c +++ kamid/client.c @@ -1616,8 +1616,14 @@ twstat(struct np_msg_header *hdr, const uint8_t *data, return; } + /* + * We skip the reading of some fields voluntarily because we + * don't support chown, so len will always be > 0! + */ +#ifdef notyet if (len != 0) goto err; +#endif if ((f = fid_by_id(fid)) == NULL) { np_error(hdr->tag, "invalid fid");