[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 15/21] s390x/cpu_models: Fix latent feature property error ha
From: |
David Hildenbrand |
Subject: |
Re: [PATCH 15/21] s390x/cpu_models: Fix latent feature property error handling bugs |
Date: |
Mon, 2 Dec 2019 10:57:44 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.1 |
On 30.11.19 20:42, Markus Armbruster wrote:
> s390x-cpu property setters set_feature() and set_feature_group() crash
> when the visitor fails and its @errp argument is null. Messed up in
> commit 0754f60429 "s390x/cpumodel: expose features and feature groups
> as properties".
Same comment as to the other patches :)
I think we usually use "s390x/cpumodels", but that's just nitpicking.
Reviewed-by: David Hildenbrand <address@hidden>
>
> The bug can't bite as no caller actually passes null. Fix it anyway.
>
> Cc: David Hildenbrand <address@hidden>
> Cc: Cornelia Huck <address@hidden>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> target/s390x/cpu_models.c | 12 ++++++++----
> 1 file changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
> index 7e92fb2e15..6a29fd3ab1 100644
> --- a/target/s390x/cpu_models.c
> +++ b/target/s390x/cpu_models.c
> @@ -987,6 +987,7 @@ static void get_feature(Object *obj, Visitor *v, const
> char *name,
> static void set_feature(Object *obj, Visitor *v, const char *name,
> void *opaque, Error **errp)
> {
> + Error *err = NULL;
> S390Feat feat = (S390Feat) opaque;
> DeviceState *dev = DEVICE(obj);
> S390CPU *cpu = S390_CPU(obj);
> @@ -1002,8 +1003,9 @@ static void set_feature(Object *obj, Visitor *v, const
> char *name,
> return;
> }
>
> - visit_type_bool(v, name, &value, errp);
> - if (*errp) {
> + visit_type_bool(v, name, &value, &err);
> + if (err) {
> + error_propagate(errp, err);
> return;
> }
> if (value) {
> @@ -1043,6 +1045,7 @@ static void get_feature_group(Object *obj, Visitor *v,
> const char *name,
> static void set_feature_group(Object *obj, Visitor *v, const char *name,
> void *opaque, Error **errp)
> {
> + Error *err = NULL;
> S390FeatGroup group = (S390FeatGroup) opaque;
> const S390FeatGroupDef *def = s390_feat_group_def(group);
> DeviceState *dev = DEVICE(obj);
> @@ -1059,8 +1062,9 @@ static void set_feature_group(Object *obj, Visitor *v,
> const char *name,
> return;
> }
>
> - visit_type_bool(v, name, &value, errp);
> - if (*errp) {
> + visit_type_bool(v, name, &value, &err);
> + if (err) {
> + error_propagate(errp, err);
> return;
> }
> if (value) {
>
--
Thanks,
David / dhildenb
- Re: [PATCH 15/21] s390x/cpu_models: Fix latent feature property error handling bugs,
David Hildenbrand <=