Your patch is correct. The call to sttrchr should be changed to afs_strchr. At some point we should think about using the native str* functions in those kernels that provide them.