erofs-utils: mkfs: add missing `errno = 0` before strto[u]l

strtoull(3) says:

```
Since strtoul() can legitimately return 0 or ULONG_MAX (ULLONG_MAX for
strtoull()) on both success and failure, the calling program should set
errno to 0 before the call, and then determine if an error occurred by
checking whether errno has a nonzero value after the call.
```

Otherwise, `--workers=` could exit with `invalid worker number`.

Fixes: 7894301e1a80 ("erofs-utils: mkfs: add `--workers=#` parameter")
Fixes: 0132cb5ea7d0 ("erofs-utils: mkfs: add `--zfeature-bits` option")
Fixes: 7550a30c332c ("erofs-utils: enable incremental builds")
Signed-off-by: Gao Xiang <hsiangkao@linux.alibaba.com>
Link: https://lore.kernel.org/r/20250214062407.3281416-1-hsiangkao@linux.alibaba.com

diff --git a/mkfs/main.c b/mkfs/main.c
index 53d330bba21871097ba863846f856912a41b77ad..b2396d016ac6447b7e5a57963b7c027ed381a5e3 100644 (file)
--- a/mkfs/main.c
+++ b/mkfs/main.c
@@ -814,6 +814,7 @@ static int mkfs_parse_options_cfg(int argc, char *argv[])
                case 520: {
                        unsigned int processors;
 
+                       errno = 0;
                        cfg.c_mt_workers = strtoul(optarg, &endptr, 0);
                        if (errno || *endptr != '\0') {
                                erofs_err("invalid worker number %s", optarg);
@@ -828,6 +829,7 @@ static int mkfs_parse_options_cfg(int argc, char *argv[])
                }
 #endif
                case 521:
+                       errno = 0;
                        i = strtol(optarg, &endptr, 0);
                        if (errno || *endptr != '\0') {
                                erofs_err("invalid zfeature bits %s", optarg);
@@ -844,6 +846,7 @@ static int mkfs_parse_options_cfg(int argc, char *argv[])
                        } else if (!strcmp(optarg, "rvsp")) {
                                dataimport_mode = EROFS_MKFS_DATA_IMPORT_RVSP;
                        } else {
+                               errno = 0;
                                dataimport_mode = strtol(optarg, &endptr, 0);
                                if (errno || *endptr != '\0') {
                                        erofs_err("invalid --%s=%s",