On Tue, Jul 11, 2023 at 08:19:50AM +0200, Hannes Reinecke wrote:
>>      for (i = 0; i < sbi->s_ndevs; i++) {
>> -            blkdev_put(FDEV(i).bdev, sbi->sb->s_type);
>> +            if (i > 0)
>> +                    blkdev_put(FDEV(i).bdev, sbi->sb->s_type);
> You could have started the loop at '1', and avoid the curious 'if' clause 

That's what the previous version did, which caused a NULL pointer
dereference discussed in this thread, as well as a compile error for
non-zoned builds..



_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to