Re: [PATCH] add checking of return status on fstat calls

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 5/15/24 8:45 AM, Mariusz Tkaczyk wrote:
On Wed, 15 May 2024 07:55:57 -0400
Nigel Croxon <ncroxon@xxxxxxxxxx> wrote:

 From 99c48231fe50845f622572d10fbb91a5ece0501f Mon Sep 17 00:00:00 2001
From: Nigel Croxon <ncroxon@xxxxxxxxxx>
Date: Fri, 10 May 2024 09:08:02 -0400
Subject: [PATCH] add checking of return status on fstat calls

There are a few places we don't check the return status when
calling fstat for success. Clean up the calls by adding a
check before continuing.

Signed-off-by: Nigel Croxon <ncroxon@xxxxxxxxxx>
---
Hi Nigel,
I'm observing following errors:

warning: Patch sent with format=flowed; space at the end of lines might be lost.
Applying: add checking of return status on fstat calls
error: corrupt patch at line 16
Patch failed at 0001 add checking of return status on fstat calls
----

Probably you need to rebase patch to current main.

I also see, some checkpatch issues like "rv=0".
At this moment, you can just fork the github repo:
https://github.com/md-raid-utilities/mdadm

and open pull request, then checkpatch will be automatically run for you :)

Thanks,
Mariusz

Hello Mariusz,

https://github.com/md-raid-utilities/mdadm/pull/7

-Nigel






[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux