On Sun, Aug 18, 2019 at 11:03:53AM +0200, Richard Weinberger wrote: > ----- Ursprüngliche Mail ----- > > I agree with you, but what can we do now is trying our best to fuzz > > all the fields. > > > > So, what is your opinion about EROFS? > > All I'm saying is that you should not blindly trust the disk. > > Another thing that raises my attention is in superblock_read(): > memcpy(sbi->volume_name, layout->volume_name, > sizeof(layout->volume_name)); > > Where do you check whether ->volume_name has a NUL terminator? > Currently this field has no user, maybe will add a check upon usage. > But this kind of things makes me wonder. You have looked at reiserfs lately, right? :) Not to say that erofs shouldn't be worked on to fix these kinds of issues, just that it's not an unheard of thing to trust the disk image. Especially for the normal usage model of erofs, where the whole disk image is verfied before it is allowed to be mounted as part of the boot process. thanks, greg k-h
