[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/6] Add "info uuid" command to monitor.
From: |
Gleb Natapov |
Subject: |
Re: [Qemu-devel] [PATCH 3/6] Add "info uuid" command to monitor. |
Date: |
Mon, 25 Aug 2008 12:46:04 +0300 |
On Sun, Aug 24, 2008 at 08:43:41PM +0300, Blue Swirl wrote:
> On 8/24/08, Gleb Natapov <address@hidden> wrote:
> > Signed-off-by: Gleb Natapov <address@hidden>
> > ---
> >
> > monitor.c | 13 +++++++++++++
> > 1 files changed, 13 insertions(+), 0 deletions(-)
> >
> > diff --git a/monitor.c b/monitor.c
> > index e71f49e..1e477bb 100644
> > --- a/monitor.c
> > +++ b/monitor.c
> > @@ -253,6 +253,17 @@ static void do_info_name(void)
> > term_printf("%s\n", qemu_name);
> > }
> >
> > +static void do_info_uuid(void)
> > +{
> > + char uuid_str[37];
> > + sprintf(uuid_str, UUID_FMT, qemu_uuid[0], qemu_uuid[1], qemu_uuid[2],
> > + qemu_uuid[3], qemu_uuid[4], qemu_uuid[5], qemu_uuid[6],
> > + qemu_uuid[7], qemu_uuid[8], qemu_uuid[9], qemu_uuid[10],
> > + qemu_uuid[11], qemu_uuid[12], qemu_uuid[13], qemu_uuid[14],
> > + qemu_uuid[15]);
> > + term_printf("%s\n", uuid_str);
>
> Please use snprintf. Or why not directly term_printf(UUID_FMT "\n",
> qemu_uuid[0] etc.?
Don't know. I've removed redundant sprintf. Thanks.
--
Gleb.
- [Qemu-devel] [PATCH 0/6] Add UUID command-line option, Gleb Natapov, 2008/08/24
- [Qemu-devel] [PATCH 1/6] Use IO port for qemu<->guest BIOS communication., Gleb Natapov, 2008/08/24
- [Qemu-devel] [PATCH 2/6] Add -uuid command line option., Gleb Natapov, 2008/08/24
- [Qemu-devel] [PATCH 3/6] Add "info uuid" command to monitor., Gleb Natapov, 2008/08/24
- [Qemu-devel] [PATCH 4/6] Use libuuid if available., Gleb Natapov, 2008/08/24
- [Qemu-devel] [PATCH 5/6] Add UUID to BIOS configuration info., Gleb Natapov, 2008/08/24
- [Qemu-devel] [PATCH 6/6] Pass cpu speed into SM BIOS., Gleb Natapov, 2008/08/24
- Re: [Qemu-devel] [PATCH 0/6] Add UUID command-line option, Blue Swirl, 2008/08/24