Re: [Qemu-devel] [PATCH 1/7] qom: add object_property_add_alias()

[Stefan Hajnoczi <stefanha@redhat.com>]

On Fri, May 23, 2014 at 12:02:44AM +1000, Peter Crosthwaite wrote:
> On Thu, May 22, 2014 at 6:22 AM, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> > +void object_property_add_alias(Object *obj, const char *name,
> > +                               Object *target_obj, const char *target_name,
> > +                               Error **errp)
> > +{
> > +    AliasProperty *prop;
> > +    ObjectProperty *target_prop;
> > +
> > +    target_prop = object_property_find(target_obj, target_name, errp);
> > +    if (!target_prop) {
> > +        return;
> 
> Not sure silent failure is right here. This should perhaps populate
> errp and then caller can decide to assert/ignore/report as needed.

object_property_find() already populates errp so the error is not
silent.  Did you miss the errp argument to object_property_find()?

Stefan