Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()

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

Subject: Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
From: Max Kellermann <max.kellermann@xxxxxxxxx>
Date: Thu, 5 Dec 2024 16:42:30 +0100
Cc: Alex Markuze <amarkuze@xxxxxxxxxx>, xiubli@xxxxxxxxxx, ceph-devel@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, stable@xxxxxxxxxxxxxxx
In-reply-to: <CAKPOu+-CpzPaY28MH9Og=mZTYmu99MUFTs+ezDZvud0HVb9PAw@mail.gmail.com>

On Thu, Dec 5, 2024 at 4:37 PM Max Kellermann <max.kellermann@xxxxxxxxx> wrote:
> btw. Alex's patch
> (https://github.com/ceph/ceph-client/commit/2a802a906f9c89f8ae4)
> introduces another memory leak

Sorry, that was wrong. The "break" only exits the inner "while" loop.
I got confused by nested loops.

References:
- [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Max Kellermann
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Alex Markuze
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Max Kellermann
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Max Kellermann
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Ilya Dryomov
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Alex Markuze
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Max Kellermann
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Max Kellermann
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Alex Markuze
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Max Kellermann
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Ilya Dryomov
- Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
  - From: Max Kellermann

Prev by Date: Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
Next by Date: Re: [PATCH 6.12 000/826] 6.12.2-rc1 review
Previous by thread: Re: [PATCH] fs/ceph/file: fix memory leaks in __ceph_sync_read()
Next by thread: [PATCH v2] fs/ceph/file: fix memory leaks in __ceph_sync_read()
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel] [Kernel Development Newbies] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Yosemite Hiking] [Linux Kernel] [Linux SCSI]