commit 8ff457f56db9f94f4baf68f53d87e791584d1dfe
from: Tim Newsham <tim.newsham@gmail.com>
date: Thu Aug 20 15:37:24 2009 UTC

lib9: dotu fixes in convS2M, convM2S

http://codereview.appspot.com/109049

commit - e7c5e5ed94e02e969d45d8ab74876cac53694195
commit + 8ff457f56db9f94f4baf68f53d87e791584d1dfe
blob - a5dae3371e3bb3431e361871b97dc69ffc842ab8
blob + 1e32b339cb59b81e481a4ea9722eea740fa7ed33
--- include/fcall.h
+++ include/fcall.h
@@ -44,6 +44,7 @@ struct	Fcall
 	
 	/* 9P2000.u extensions */
 	int	errornum;	/* Rerror */
+	int	uidnum;		/* Tattach, Tauth */
 	char	*extension;	/* Tcreate */
 } Fcall;
 
@@ -73,6 +74,7 @@ struct	Fcall
 
 #define	NOTAG		(ushort)~0U	/* Dummy tag */
 #define	NOFID		(u32int)~0U	/* Dummy fid */
+#define	NOUID		(-1)	/* Dummy uid */
 #define	IOHDRSZ		24	/* ample room for Twrite/Rread header (iounit) */
 
 enum
blob - 3df83da2d38a93ca04a58e78258d4c69b8268d08
blob + 6a8805c4d6ac83dd5b9a3efb3aa3c6818212433b
--- src/lib9/convM2S.c
+++ src/lib9/convM2S.c
@@ -100,6 +100,13 @@ convM2Su(uchar *ap, uint nap, Fcall *f, int dotu)
 		p = gstring(p, ep, &f->aname);
 		if(p == nil)
 			break;
+		f->uidnum = NOUID;
+		if(dotu){
+			if(p+BIT32SZ > ep)
+				return 0;
+			f->uidnum = GBIT32(p);
+			p += BIT32SZ;
+		}
 		break;
 
 	case Tattach:
@@ -117,6 +124,13 @@ convM2Su(uchar *ap, uint nap, Fcall *f, int dotu)
 		p = gstring(p, ep, &f->aname);
 		if(p == nil)
 			break;
+		f->uidnum = NOUID;
+		if(dotu){
+			if(p+BIT32SZ > ep)
+				return 0;
+			f->uidnum = GBIT32(p);
+			p += BIT32SZ;
+		}
 		break;
 
 	case Twalk:
@@ -233,10 +247,10 @@ convM2Su(uchar *ap, uint nap, Fcall *f, int dotu)
 		p = gstring(p, ep, &f->ename);
 		f->errornum = 0;
 		if(dotu){
-			if(p+BIT16SZ > ep)
+			if(p+BIT32SZ > ep)
 				return 0;
-			f->errornum = GBIT16(p);
-			p += BIT16SZ;
+			f->errornum = GBIT32(p);
+			p += BIT32SZ;
 		}
 		break;
 
blob - b01a022165555faa63555defc0575a03e5bad9eb
blob + 02923bd4efb06d6726645b95623d5dd679e19a66
--- src/lib9/convS2M.c
+++ src/lib9/convS2M.c
@@ -74,6 +74,8 @@ sizeS2Mu(Fcall *f, int dotu)
 		n += BIT32SZ;
 		n += stringsz(f->uname);
 		n += stringsz(f->aname);
+		if(dotu)
+			n += BIT32SZ;
 		break;
 
 	case Tattach:
@@ -81,6 +83,8 @@ sizeS2Mu(Fcall *f, int dotu)
 		n += BIT32SZ;
 		n += stringsz(f->uname);
 		n += stringsz(f->aname);
+		if(dotu)
+			n += BIT32SZ;
 		break;
 
 	case Twalk:
@@ -144,7 +148,7 @@ sizeS2Mu(Fcall *f, int dotu)
 	case Rerror:
 		n += stringsz(f->ename);
 		if(dotu)
-			n += BIT16SZ;
+			n += BIT32SZ;
 		break;
 
 	case Rflush:
@@ -249,6 +253,11 @@ convS2Mu(Fcall *f, uchar *ap, uint nap, int dotu)
 		p += BIT32SZ;
 		p  = pstring(p, f->uname);
 		p  = pstring(p, f->aname);
+		if(dotu){
+			f->uidnum = NOUID;
+			PBIT32(p, f->uidnum);
+			p += BIT32SZ;
+		}
 		break;
 
 	case Tattach:
@@ -258,6 +267,11 @@ convS2Mu(Fcall *f, uchar *ap, uint nap, int dotu)
 		p += BIT32SZ;
 		p  = pstring(p, f->uname);
 		p  = pstring(p, f->aname);
+		if(dotu){
+			f->uidnum = NOUID;
+			PBIT32(p, f->uidnum);
+			p += BIT32SZ;
+		}
 		break;
 
 	case Twalk:
@@ -344,8 +358,8 @@ convS2Mu(Fcall *f, uchar *ap, uint nap, int dotu)
 	case Rerror:
 		p = pstring(p, f->ename);
 		if(dotu){
-			PBIT16(p, f->errornum);
-			p += BIT16SZ;
+			PBIT32(p, f->errornum);
+			p += BIT32SZ;
 		}
 		break;