Jan Kryl wrote: > Hi, > > please could you review the following fix for 6920999? > > webrev: > http://cr.opensolaris.org/~jkryl/nfs-utf8/
Hi Jan, The changes look good to me except for something related to the ASSERT change Tom mentioned. I think ASSERTs and a runtime check should be put in nfscmd_findmap(), which is the highest point that is vulnerable to a stray NULL pointer. Thanks for running with this, I had not appreciated there was incomplete implementation on the server as well. The CR should be clear that this problem and this fix is about the server side, not the client. Rob T
