Changeset 15e51b05 in mainline

Timestamp:

2025-07-07T11:04:41Z (3 months ago)

Author:

Miroslav Cimerman <mc@…>

Children:

9323bb8

Parents:

1162b6c

Message:

hr: check extent number

Location:

uspace/srv/bd/hr

Files:

• hr.c (modified) (1 diff)
• metadata/foreign/geom/hr_g_stripe.c (modified) (1 diff)
• metadata/native.c (modified) (1 diff)

uspace/srv/bd/hr/hr.c

@@ -110 +110 @@
                 async_answer_0(&call, rc);
                 async_answer_0(icall, rc);
+                return;
+        }
+
+        if (cfg->dev_no > HR_MAX_EXTENTS) {
+                HR_ERROR("provided %u devices (max = %u)",
+                    (unsigned)cfg->dev_no, HR_MAX_EXTENTS);
+                free(cfg);
+                async_answer_0(icall, ELIMIT);
                 return;
         }

uspace/srv/bd/hr/metadata/foreign/geom/hr_g_stripe.c

@@ -168 +168 @@
         vol->data_offset = 0;
 
-        if (main_meta->md_all > HR_MAX_EXTENTS) {
+        if (vol->extent_no > HR_MAX_EXTENTS) {
                 HR_DEBUG("Assembled volume has %u extents (max = %u)",
                     (unsigned)main_meta->md_all, HR_MAX_EXTENTS);

uspace/srv/bd/hr/metadata/native.c

@@ -189 +189 @@
         /* memcpy(vol->devname, main_meta->devname, HR_DEVNAME_LEN); */
 
+        if (vol->extent_no > HR_MAX_EXTENTS) {
+                HR_DEBUG("Assembled volume has %u extents (max = %u)",
+                    (unsigned)vol->extent_no, HR_MAX_EXTENTS);
+                return EINVAL;
+        }
+
         vol->in_mem_md = calloc(1, sizeof(hr_metadata_t));
         if (vol->in_mem_md == NULL)