* [Qemu-devel] qemu monitor.c vl.c
@ 2006-07-14 20:26 Fabrice Bellard
2006-07-15 21:18 ` [Qemu-devel] " Anthony Liguori
0 siblings, 1 reply; 2+ messages in thread
From: Fabrice Bellard @ 2006-07-14 20:26 UTC (permalink / raw)
To: qemu-devel
CVSROOT: /sources/qemu
Module name: qemu
Changes by: Fabrice Bellard <bellard> 06/07/14 20:26:42
Modified files:
. : monitor.c vl.c
Log message:
generate CRLF instead of LF
CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/qemu/monitor.c?cvsroot=qemu&r1=1.51&r2=1.52
http://cvs.savannah.gnu.org/viewcvs/qemu/vl.c?cvsroot=qemu&r1=1.199&r2=1.200
^ permalink raw reply [flat|nested] 2+ messages in thread
* [Qemu-devel] Re: qemu monitor.c vl.c
2006-07-14 20:26 [Qemu-devel] qemu monitor.c vl.c Fabrice Bellard
@ 2006-07-15 21:18 ` Anthony Liguori
0 siblings, 0 replies; 2+ messages in thread
From: Anthony Liguori @ 2006-07-15 21:18 UTC (permalink / raw)
To: qemu-devel
I think you want to do check to see if the term buffer is full after
writing the '\r' here right? Otherwise, I presume it's possible to get a
buffer overflow is '\n' would normally fill the buffer.
Regards,
Anthony Liguori
On Fri, 14 Jul 2006 20:26:42 +0000, Fabrice Bellard wrote:
> CVSROOT: /sources/qemu
> Module name: qemu
> Changes by: Fabrice Bellard <bellard> 06/07/14 20:26:42
>
> Modified files:
> . : monitor.c vl.c
>
> Log message:
> generate CRLF instead of LF
>
> CVSWeb URLs:
> http://cvs.savannah.gnu.org/viewcvs/qemu/monitor.c?cvsroot=qemu&r1=1.51&r2=1.52
> http://cvs.savannah.gnu.org/viewcvs/qemu/vl.c?cvsroot=qemu&r1=1.199&r2=1.200
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2006-07-15 21:23 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-07-14 20:26 [Qemu-devel] qemu monitor.c vl.c Fabrice Bellard
2006-07-15 21:18 ` [Qemu-devel] " Anthony Liguori
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).