ufs: Allow fid to be 0

titanous · progrium · commit e571b90ca9a0 · 2017-06-20T17:56:40.000-07:00
0 is a valid fid, so check against the NOFID value instead.

Signed-off-by: Jonathan Rudenberg &lt;jonathan@titanous.com&gt;
diff --git a/ufs/session.go b/ufs/session.go
@@ -31,7 +31,7 @@ func (sess *session) getRef(fid p9p.Fid) (*FileRef, error) {
 	sess.Lock()
 	defer sess.Unlock()
 
-	if fid == 0 {
+	if fid == p9p.NOFID {
 		return nil, p9p.ErrUnknownfid
 	}
 
@@ -51,7 +51,7 @@ func (sess *session) newRef(fid p9p.Fid, path string) (*FileRef, error) {
 	sess.Lock()
 	defer sess.Unlock()
 
-	if fid == 0 {
+	if fid == p9p.NOFID {
 		return nil, p9p.ErrUnknownfid
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ func (sess session) getRef(fid p9p.Fid) (FileRef, error) {`
`31`	`31`	`sess.Lock()`
`32`	`32`	`defer sess.Unlock()`
`33`	`33`
`34`		`- if fid == 0 {`
	`34`	`+ if fid == p9p.NOFID {`
`35`	`35`	`return nil, p9p.ErrUnknownfid`
`36`	`36`	`}`
`37`	`37`
`@@ -51,7 +51,7 @@ func (sess session) newRef(fid p9p.Fid, path string) (FileRef, error) {`
`51`	`51`	`sess.Lock()`
`52`	`52`	`defer sess.Unlock()`
`53`	`53`
`54`		`- if fid == 0 {`
	`54`	`+ if fid == p9p.NOFID {`
`55`	`55`	`return nil, p9p.ErrUnknownfid`
`56`	`56`	`}`
`57`	`57`