From: Karel Zak <kzak@redhat.com>
To: Sami Kerola <kerolasa@iki.fi>
Cc: util-linux@vger.kernel.org
Subject: Re: [PATCH 00/34] pull: make printing quicker, and clang analysis
Date: Tue, 9 Jul 2013 14:58:17 +0200 [thread overview]
Message-ID: <20130709125817.GD24716@x2.net.home> (raw)
In-Reply-To: <1373226889-3451-1-git-send-email-kerolasa@iki.fi>
On Sun, Jul 07, 2013 at 08:54:15PM +0100, Sami Kerola wrote:
> sfdisk: make unhiding as complete as possible
> docs: add missing options to sfdisk manual
Applied.
> include: copy unlocked-io.h from gnulib
> hexdump: use unlocked io, and avoid use of printf()
> text-utils: use unlocked io
> misc-utils: use unlocked io
> scriptreplay: use unlocked io
> dmesg: use unlocked io, and avoid use of printf()
> utmpdump: use unlocked io, and avoid use of printf()
Postponed.
> dmesg: inform user --show-delta and iso8601 time format does not mix
> lib/loopdev: assigned value is never read [clang-analyzer]
> lib/mbsalign: initializations values are never read [clang-analyzer]
> libmount: fix memory leak [clang-analyzer]
Fixed and applied.
> ipcs: assigned values are never read [clang-analyzer]
> lscpu: fix memory leak [clang-analyzer]
> more: assigned value is never read [clang-analyzer]
> utmpdump: assigned values are never read [clang-analyzer]
> mkfs.cramfs: argument to free() is a constant address [clang-analyzer]
> eject: assigned value is never read [clang-analyzer]
> chfn: assigned value is never read [clang-analyzer]
Applied.
> mesg: assigned value is never read [clang-analyzer]
> rev: assigned value is never read [clang-analyzer]
> column: assigned value is never read [clang-analyzer]
> hwclock: assigned value is never read [clang-analyzer]
> dmesg: assigned value is never read [clang-analyzer]
> login: assigned value is never read [clang-analyzer]
> fdisk: assigned value is never read [clang-analyzer]
Ignored as keep argv and argc consistent seems better.
> sulogin: fix memory leak [clang-analyzer]
> agetty: fix memory leak [clang-analyzer]
> dmesg: add missing initializer [clang-analyzer]
^^^^^
uuidd:
Fixed & applied.
> ul: use correct types
> ul: use string printing function
> column: dereference of null pointer [clang-analyzer]
> more: use variable lenght printf field width to print blanks
Applied (with some changes).
Karel
--
Karel Zak <kzak@redhat.com>
http://karelzak.blogspot.com
prev parent reply other threads:[~2013-07-09 12:58 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-07 19:54 [PATCH 00/34] pull: make printing quicker, and clang analysis Sami Kerola
2013-07-07 19:54 ` [PATCH 01/34] sfdisk: make unhiding as complete as possible Sami Kerola
2013-07-07 19:54 ` [PATCH 02/34] docs: add missing options to sfdisk manual Sami Kerola
2013-07-07 19:54 ` [PATCH 03/34] include: copy unlocked-io.h from gnulib Sami Kerola
2013-07-08 20:21 ` Kay Sievers
2013-07-09 6:37 ` Sami Kerola
2013-07-09 9:16 ` Kay Sievers
2013-07-13 22:10 ` Sami Kerola
2013-07-07 19:54 ` [PATCH 04/34] hexdump: use unlocked io, and avoid use of printf() Sami Kerola
2013-07-07 19:54 ` [PATCH 05/34] text-utils: use unlocked io Sami Kerola
2013-07-07 19:54 ` [PATCH 06/34] misc-utils: " Sami Kerola
2013-07-07 19:54 ` [PATCH 07/34] scriptreplay: " Sami Kerola
2013-07-07 19:54 ` [PATCH 09/34] utmpdump: use unlocked io, and avoid use of printf() Sami Kerola
2013-07-07 19:54 ` [PATCH 10/34] dmesg: inform user --show-delta and iso8601 time format does not mix Sami Kerola
2013-07-07 19:54 ` [PATCH 11/34] lib/loopdev: assigned value is never read [clang-analyzer] Sami Kerola
2013-07-07 19:54 ` [PATCH 12/34] lib/mbsalign: initializations values are " Sami Kerola
2013-07-07 19:54 ` [PATCH 13/34] libmount: fix memory leak [clang-analyzer] Sami Kerola
2013-07-08 8:02 ` Karel Zak
2013-07-07 19:54 ` [PATCH 14/34] ipcs: assigned values are never read [clang-analyzer] Sami Kerola
2013-07-07 19:54 ` [PATCH 15/34] lscpu: fix memory leak [clang-analyzer] Sami Kerola
2013-07-07 19:54 ` [PATCH 16/34] more: assigned value is never read [clang-analyzer] Sami Kerola
2013-07-07 19:54 ` [PATCH 17/34] utmpdump: assigned values are " Sami Kerola
2013-07-07 19:54 ` [PATCH 18/34] mkfs.cramfs: argument to free() is a constant address [clang-analyzer] Sami Kerola
2013-07-07 19:54 ` [PATCH 19/34] eject: assigned value is never read [clang-analyzer] Sami Kerola
2013-07-07 19:54 ` [PATCH 20/34] chfn: " Sami Kerola
2013-07-07 19:54 ` [PATCH 21/34] mesg: " Sami Kerola
2013-07-08 8:23 ` Karel Zak
2013-07-07 19:54 ` [PATCH 22/34] rev: " Sami Kerola
2013-07-07 19:54 ` [PATCH 23/34] column: " Sami Kerola
2013-07-07 19:54 ` [PATCH 24/34] hwclock: " Sami Kerola
2013-07-07 19:54 ` [PATCH 25/34] dmesg: " Sami Kerola
2013-07-07 19:54 ` [PATCH 26/34] login: " Sami Kerola
2013-07-07 19:54 ` [PATCH 27/34] sfdisk: " Sami Kerola
2013-07-07 19:54 ` [PATCH 28/34] sulogin: fix memory leak [clang-analyzer] Sami Kerola
2013-07-07 19:54 ` [PATCH 29/34] agetty: " Sami Kerola
2013-07-07 19:54 ` [PATCH 30/34] dmesg: add missing initializer [clang-analyzer] Sami Kerola
2013-07-07 19:54 ` [PATCH 31/34] ul: use correct types Sami Kerola
2013-07-07 19:54 ` [PATCH 32/34] ul: use string printing function Sami Kerola
2013-07-07 19:54 ` [PATCH 33/34] column: dereference of null pointer [clang-analyzer] Sami Kerola
2013-07-07 19:54 ` [PATCH 34/34] more: use variable lenght printf field width to print blanks Sami Kerola
2013-07-09 12:58 ` Karel Zak [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130709125817.GD24716@x2.net.home \
--to=kzak@redhat.com \
--cc=kerolasa@iki.fi \
--cc=util-linux@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox