Commit Diff

Commit:
f4ad0146e973695127d1c9e1769cd2a7882184d8
From:
Omar Polo <op@omarpolo.com>
Date:
Message:
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!
Actions:
Patch | Tree
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");