| Message ID | 20231023095549.719099-2-twuufnxlz@gmail.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net] ptp: ptp_read should not release queue | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Clearly marked for net |
| netdev/apply | fail | Patch does not apply to net |
On Mon, 23 Oct 2023 17:55:50 +0800 Edward AD wrote:
> Signed-off-by: Edward AD <twuufnxlz@gmail.com>
We need a legal name for the signoff, not initials.
On Mon, 23 Oct 2023 09:33:34 -0700 Jakub Kicinski wrote: >> Signed-off-by: Edward AD <twuufnxlz@gmail.com> > > We need a legal name for the signoff, not initials. You may have some misunderstandings about my name, AD is not an acronym. This is my full and legal name 'Edward AD'.
On Tue, 24 Oct 2023 08:34:58 +0800 Edward AD wrote: > > We need a legal name for the signoff, not initials. > You may have some misunderstandings about my name, AD is not an acronym. > This is my full and legal name 'Edward AD'. Is there any public person with the surname 'AD' you can point me to? Or any reference that would educate me about it?
diff --git a/drivers/ptp/ptp_chardev.c b/drivers/ptp/ptp_chardev.c index 282cd7d24077..27c1ef493617 100644 --- a/drivers/ptp/ptp_chardev.c +++ b/drivers/ptp/ptp_chardev.c @@ -585,7 +585,5 @@ ssize_t ptp_read(struct posix_clock_context *pccontext, uint rdflags, free_event: kfree(event); exit: - if (result < 0) - ptp_release(pccontext); return result; }
Firstly, queue is not the memory allocated in ptp_read; Secondly, other processes may block at ptp_read and wait for conditions to be met to perform read operations. Reported-and-tested-by: syzbot+9704e6f099d952508943@syzkaller.appspotmail.com Fixes: 8f5de6fb2453 ("ptp: support multiple timestamp event readers") Signed-off-by: Edward AD <twuufnxlz@gmail.com> --- drivers/ptp/ptp_chardev.c | 2 -- 1 file changed, 2 deletions(-)