> > @@ -733,6 +734,9 @@ > > > > fb_delete(fbuf); > > } > > + > > + if (!error && ap->a_eofflag != NULL) > > + *ap->a_eofflag = eofflag; > > Is the null check here necessary? Other file systems don't do this, so I'm > wondering if you encountered a null pointer here. I didn't encounter a null pointer and none of the callers pass NULL as a eofflag here, so I guess I included the check out of sheer paranoia. ;)
cheers, natano
