Re: [PATCH 00/15] uuidd: refactoring & systemd support (take 2)

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

 



On Thu, May 03, 2012 at 09:01:46PM +0200, Petr Uzel wrote:
> Regarding the last patch in the series, it addresses issue described in
> http://marc.info/?l=util-linux-ng&m=133405763330121&w=2 in the way recomended
> by Karel. However, after implementing it, I'm wondering: wouldn't it be cleaner
> to just test if /dev/u?random exists and is readable in uuid_generate() instead
> of calling random_get_fd() and passing the fd down the stack?

 random_get_fd() does much more than only open the file

>  Makefile.am                 |    3 +-
>  configure.ac                |   23 ++
>  fdisk/fdiskdoslabel.c       |    2 +-
>  include/randutils.h         |    2 +-
>  lib/randutils.c             |    8 +-
>  libuuid/src/Makefile.am     |    1 -
>  libuuid/src/gen_uuid.c      |   65 +-----
>  libuuid/src/uuidd.h         |    2 +-
>  misc-utils/.gitignore       |    1 +
>  misc-utils/Makefile.am      |   17 ++-
>  misc-utils/sd-daemon.c      |  530 +++++++++++++++++++++++++++++++++++++++++++
>  misc-utils/sd-daemon.h      |  282 +++++++++++++++++++++++
>  misc-utils/uuidd.8          |   12 +
>  misc-utils/uuidd.c          |  313 +++++++++++++++++---------
>  misc-utils/uuidd.service.in |    9 +
>  misc-utils/uuidd.socket     |    8 +
>  16 files changed, 1109 insertions(+), 169 deletions(-)
>  create mode 100644 misc-utils/sd-daemon.c
>  create mode 100644 misc-utils/sd-daemon.h
>  create mode 100644 misc-utils/uuidd.service.in
>  create mode 100644 misc-utils/uuidd.socket

 Applied, thanks a lot!

    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