shejialuo <shejialuo@xxxxxxxxx> writes:
[snip]
> + if (strbuf_read_file(&ref_content, iter->path.buf, 0) < 0) {
> + ret = fsck_report_ref(o, &report,
> + FSCK_MSG_BAD_REF_CONTENT,
> + "cannot read ref file");
> + goto cleanup;
> + }
> +
Shouldn't we use `die_errno` here instead? I mean, this is not really a
bad ref content issue. If we don't want to die here, it would still
probably be nice to get the actual issue using `strerror` instead and
use that instead of the generic message we have here.
[snip]
Attachment:
signature.asc
Description: PGP signature
