On Fri, 26 Apr 2013 16:46:57 +0200 Stefan Hajnoczi <stefa...@gmail.com> wrote:
> On Fri, Apr 26, 2013 at 05:31:15PM +0800, Wenchao Xia wrote: > > @@ -2586,10 +2585,12 @@ void do_info_snapshots(Monitor *mon, const QDict > > *qdict) > > } > > > > if (total > 0) { > > - monitor_printf(mon, "%s\n", bdrv_snapshot_dump(buf, sizeof(buf), > > NULL)); > > + bdrv_snapshot_dump(NULL); > > + monitor_printf(mon, "\n"); > > Luiz: any issue with mixing monitor_printf(mon) and > monitor_vprintf(cur_mon) calls? I guess there was a reason for > explicitly passing mon instead of relying on cur_mon. where are they being mixed?