diff options
| author | Marc-André Lureau <marcandre.lureau@gmail.com> | 2012-03-20 21:38:33 +0100 |
|---|---|---|
| committer | Marc-André Lureau <marcandre.lureau@gmail.com> | 2012-03-21 13:24:17 +0100 |
| commit | feaee860768c0a815f0e2e98d6fd51e6fff4321f (patch) | |
| tree | bcdca9f01fd5d0f9d918c5ce2781f3dc4bd2d632 | |
| parent | bc4f5db323266cf7ac3f1ac6dd15509778cd3bf4 (diff) | |
spice.proto: fix demarshaller crash with name message
It turned out the demarshaller wasn't allocating enough space to
memcpy the name. In order to take into account the size of a variable
array, it needs to be marked with the @end tag so that the
"extra_size" is added to the allocated memory.
It would be nice if the demarshaller would somehow fail if this wasn't
set explicitly, or do the right thing by default.
@end the name so that demarshaller
| -rw-r--r-- | spice.proto | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spice.proto b/spice.proto index e3feffa..513fe87 100644 --- a/spice.proto +++ b/spice.proto @@ -225,7 +225,7 @@ channel MainChannel : BaseChannel { message { uint32 name_len; - uint8 name[name_len]; + uint8 name[name_len] @end; } name; message { |