On 2020.06.14 08:23, Thomas Schmitt wrote:
This would be clearer if the test would be in the reuse case of/* Reuse multiextent p_stat if not NULL */ if (!p_stat) { p_stat = calloc(1, stat_len); first_extent = true; } else { + /* Ignore Rock Ridge Deep Directory RE entries */ + if (p_stat->rr.u_su_fields & ISO_ROCK_SUF_RE) + goto fail; first_extent = false; }
I agree. I'll apply this (and validate that it doesn't break anything) before merging to mainline.
Regards, /Pete