On 20/07/2020 14:34, Damien Le Moal wrote:
[...]
>> + zone->capacity += next->capacity;
>> if (next->cond == BLK_ZONE_COND_READONLY &&
>> zone->cond != BLK_ZONE_COND_OFFLINE)
>> zone->cond = BLK_ZONE_COND_READONLY;
>> else if (next->cond == BLK_ZONE_COND_OFFLINE)
>> zone->cond = BLK_ZONE_COND_OFFLINE;
>> }
>> + if (zone->capacity != zone->len) {
>> + zonefs_err(sb, "Invalid conventional zone capacity\n");
>> + ret = -EINVAL;
>
> Need "goto free;" here. Forgot it too in the code snippet I sent..
>
Oh right sorry for missing it.
