Re: [PATCH 0/6] fix various minor issues and warnings

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

 



On Tue, Jun 13, 2017 at 05:18:33PM +0200, Rüdiger Meier wrote:
> On 06/13/2017 03:47 PM, Karel Zak wrote:
> > On Mon, Jun 12, 2017 at 10:49:52AM +0200, Ruediger Meier wrote:
> > > From: Ruediger Meier <ruediger.meier@xxxxxxxxxxx>
> > > 
> > > Ruediger Meier (6):
> > >    tests: fix fincore, don't use variable COLUMNS
> > >    libmount: btrfs, remove unused setter functions
> > >    hwclock: fix warning [-Winvalid-noreturn]
> > >    misc: fix gcc-7 sprintf warnings -Wformat-overflow
> > >    misc: fix gcc-7 snprintf warnings -Wformat-truncation
> > >    libblkid: fix gcc-7 warning -Wint-in-bool-context
> > > 
> > >   lib/loopdev.c                    |  2 +-
> > >   lib/sysfs.c                      |  4 ++--
> > >   libblkid/src/devname.c           |  2 +-
> > >   libblkid/src/superblocks/exfat.c |  4 ++--
> > >   libmount/src/btrfs.c             |  4 ----
> > >   login-utils/last.c               |  2 +-
> > >   sys-utils/hwclock.c              |  2 +-
> > >   sys-utils/hwclock.h              |  3 ++-
> > >   tests/ts/fincore/count           | 12 ++++++------
> > >   9 files changed, 16 insertions(+), 19 deletions(-)
> > 
> > Seems better than the previous version from Sami (May 27).
> 
> These fixed are completely independent/unrelated to the other pull
> request from Sami. Sami's PR just motivated me to try out gcc-7 too.

Yep, you both tried to fix the sprintf warnings, your version with
NAME_MAX seems better.

Applied, thanks.

    Karel

-- 
 Karel Zak  <kzak@xxxxxxxxxx>
 http://karelzak.blogspot.com
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux