Re: [Qemu-devel] [PATCH v2 08/27] qom: fix off-by-one

[Anthony Liguori]

On 02/04/2012 02:02 AM, Paolo Bonzini wrote:
Signed-off-by: Paolo Bonzini<pbonz...@redhat.com>
---
  qom/object.c |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/qom/object.c b/qom/object.c
index b26272f..314fc7a 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -871,7 +871,7 @@ static void object_set_link_property(Object *obj, Visitor 
*v, void *opaque,
              gchar *target_type;

              target_type = g_strdup(&type[5]);
-            target_type[strlen(target_type) - 2] = 0;
+            *strchr(target_type, '>') = 0;

Should use an intermediate variable here and do a NULL check.

My eyes can't handle dereferencing strchr() directly even if I understand why 
it's safe to do :-)

Regards,

Anthony Liguori


              if (object_dynamic_cast(target, target_type)) {
                  object_ref(target);