[PATCH] erofs-utils: lib: fix QPL job leak on early error paths in z_erofs_decompress_qpl() After z_erofs_qpl_get_job() succeeds, two early-return error paths bypass z_erofs_qpl_put_job(), leaking the QPL job handle: - Line 200: return -EFSCORRUPTED (when inputmargin >= inputsize) - Line 205: return -ENOMEM (when malloc fails for decodedskip buffer) Fix by replacing the bare returns with goto out_inflate_end, which already handles both z_erofs_qpl_put_job() and free(buff).

[zhaoyifan (H)]

Hi Vishwakarma,


Just a quick note for you and all the beginning committers:

Before submitting any patch to our community mailing list, please 
carefully read the contribution guidelines (we follow the Linux kernel 
process: [1]).

You're welcome to use AI agents to help check whether the patch obeys 
the guidelines, but please remember to personally review your changes, 
test locally, and understand what you're submitting.


Also we are about to publish specific guidelines for AI-assisted content.


Finally, before sending any email to the mailing list, we strongly 
recommend sending a test to your own inbox first—messages posted to the 
list are permanent and cannot be edited or retracted.


[1] https://docs.kernel.org/process/submitting-patches.html

[ This email is generated with help with Qwen3.5-plus: translation from 
Chinese to English ]


Thanks,

Yifan Zhao


On 2026/3/20 10:04, Shubham Vishwakarma wrote:
> Hi Gao,
>
> I understand the concern, but I want to be honest: if I had used AI to 
> draft the initial submission, it likely would have been sent without 
> those formatting errors.
>
> The issues with the invalid email address and the oversized subject 
> line occurred because I was unfamiliar with sending via the SMTP CLI. 
> I used GPT to help me configure the CLI tool, which led to the 
> mistakes you saw.
>
> I appreciate the feedback on the patch. If there is still doubt 
> regarding my work, I will continue to submit more patches to 
> demonstrate my ability.
>
> Thanks,
>
> Shubham Vishwakarma
>
>
> On Fri, Mar 20, 2026 at 7:11 AM Gao Xiang 
> <hsiangkao at linux.alibaba.com> wrote:
>
>
>
>     On 2026/3/20 09:37, Gao Xiang wrote:
>     >
>     >
>     > On 2026/3/20 09:16, Ajay Rajera wrote:
>     >> Hi Vi-shub,
>     >> just a review :
>     >> I think the fix looks correct and it is the right approach but the
>     >> commit message formatting needs work. The entire description is
>     in the
>     >> subject line. Per kernel conventions, the subject should be a short
>     >> one-liner, e.g: erofs-utils: lib: fix QPL job leak on early error
>     >> paths
>     >> The detailed explanation (which error paths leak, why, and how
>     the fix
>     >> works) should go in the commit message body, separated from the
>     >> subject.
>     >>
>     >> So you can resend with the subject/body split fixed? so It will
>     look more clear.
>     >
>     > yes, the commit message is in a mess.
>
>     BTW, I don't know who is using the email address
>     <yifan at pku.edu.cn>, and the recipient doesn't exist.
>
>     Here, I want to say, I have to identfy you as another
>     AI bot.
>
>     Thanks,
>     Gao Xiang
>
>     >
>     > Thanks,
>     > Gao Xiang
>     >
>     >> Thanks, Ajay
>     >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/linux-erofs/attachments/20260320/95e9371d/attachment.htm>