On Mon, Oct 02, 2023 at 16:51:30 +0000, Lidong Chen wrote: > The changes for the extent parsing looks good to me. But, I am not sure about > this change: > > > - if (filename + direntry->len - 1 > (char *) tail) > > + if (filename + direntry->len - 1 > (char *) end) > > Looking at the grub_xfs_next_de(), should the ‘+1’ be ‘+3’? >
If you don't mind, I think this would be better discussed on grub-devel. I only posted the update here because my original patch was linked here. -- Jon Doge Wrangler X(7): A program for managing terminal windows. See also screen(1) and tmux(1).
