On Mon, Oct 27, 2008 at 23:43:40 +0200, Guillem Jover wrote: > + > + p = grub_strchr (p, '}') + 1; > + if (p == NULL) > + goto pvs_fail; > + This test seems buggy. The +1 needs to be after the NULL check. Cheers, Julien