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

Reply via email to