[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#44235: [PATCH] dd: drop old workaround for lseek() bug in Linux kern
From: |
Jim Meyering |
Subject: |
bug#44235: [PATCH] dd: drop old workaround for lseek() bug in Linux kernel |
Date: |
Mon, 26 Oct 2020 07:40:52 -0700 |
On Mon, Oct 26, 2020 at 6:13 AM Pádraig Brady <P@draigbrady.com> wrote:
> On 26/10/2020 10:44, Kamil Dudka wrote:
> > The workaround triggers warnings with new kernel versions in case
> > a user does not have sufficient privileges for the MTIOCGET ioctl.
> >
> > * src/dd.c (skip_via_lseek): Drop wrapper function no longer needed.
> > (skip): Use lseek() directly.
> > (advance_input_after_read_error): Use lseek() directly.
> >
> > Reported-by: Nir Soffer
> > Bug: https://bugzilla.redhat.com/1876840
>
> To clarify, the kernel is warning, not dd.
> In general the kernel should be disallowing rather than warning I think.
> But yes this is old code that should no longer be needed.
> I'll apply the patch.
Thanks. Glad to see that code can be removed.
I still have that tape drive, but when I added that code (2000) may
have been the last time it was powered on.