Re: [Qemu-devel] [PATCH] monitor: assert monitor_puts()'s loop invariant

[Luiz Capitulino]

On Thu, 10 Jan 2013 14:10:22 +0100
Markus Armbruster <arm...@redhat.com> wrote:

> Chiefly to hush up Coverity.
> 
> Signed-off-by: Markus Armbruster <arm...@redhat.com>

Applied to the qmp branch, thanks.

> ---
>  monitor.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/monitor.c b/monitor.c
> index 9cf419b..c6eac60 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -270,6 +270,7 @@ static void monitor_puts(Monitor *mon, const char *str)
>      char c;
>  
>      for(;;) {
> +        assert(mon->outbuf_index < sizeof(mon->outbuf) - 1);
>          c = *str++;
>          if (c == '\0')
>              break;