Re: [PATCH 11/33] fix: IncrementalRemove leaves open handle

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

 



On Mon, 5 Jul 2010 10:23:52 +0100
"Czarnowska, Anna" <anna.czarnowska@xxxxxxxxx> wrote:

> From: Przemyslaw Czarnowski <przemyslaw.hawrylewicz.czarnowski@xxxxxxxxx>
> 
> 
> 
> Signed-off-by: Przemyslaw Czarnowski <przemyslaw.hawrylewicz.czarnowski@xxxxxxxxx<mailto:przemyslaw.hawrylewicz.czarnowski@xxxxxxxxx>>
> 
> ---
> 
>  Incremental.c |    5 ++++-
> 
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> 
> 
> diff --git a/Incremental.c b/Incremental.c index 3501bc1..4455a89 100644
> 
> --- a/Incremental.c
> 
> +++ b/Incremental.c
> 
> @@ -866,6 +866,7 @@ int Incremental_container(struct supertype *st, char *devname, int verbose,  int IncrementalRemove(char *devname, int verbose)  {
> 
>       int mdfd;
> 
> +     int rv;
> 
>       struct mdstat_ent *ent;
> 
>       struct mddev_dev_s devlist;
> 
> 
> 
> @@ -890,7 +891,9 @@ int IncrementalRemove(char *devname, int verbose)
> 
>       devlist.disposition = 'f';
> 
>       Manage_subdevs(ent->dev, mdfd, &devlist, verbose);
> 
>       devlist.disposition = 'r';
> 
> -     return Manage_subdevs(ent->dev, mdfd, &devlist, verbose);
> 
> +     rv = Manage_subdevs(ent->dev, mdfd, &devlist, verbose);
> 
> +     close(mdfd);
> 
> +     return rv;

All of these patches are formatted rather badly with lots of blank lines.

However this is simple and correct so I have applied this one by hand.

Thanks,
NeilBrown


> 
>  }
> 
> 
> 
>  /*
> 
> --
> 
> 1.6.4.2
> 
> 
> 
> ---------------------------------------------------------------------
> Intel Technology Poland sp. z o.o.
> z siedziba w Gdansku
> ul. Slowackiego 173
> 80-298 Gdansk
> 
> Sad Rejonowy Gdansk Polnoc w Gdansku, 
> VII Wydzial Gospodarczy Krajowego Rejestru Sadowego, 
> numer KRS 101882
> 
> NIP 957-07-52-316
> Kapital zakladowy 200.000 zl
> 
> This e-mail and any attachments may contain confidential material for
> the sole use of the intended recipient(s). Any review or distribution
> by others is strictly prohibited. If you are not the intended
> recipient, please contact the sender and delete all copies.

--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[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