From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail1.bemta25.messagelabs.com ([195.245.230.129]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1fuXtG-00020M-Hc for barebox@lists.infradead.org; Tue, 28 Aug 2018 06:54:18 +0000 From: Date: Tue, 28 Aug 2018 06:53:56 +0000 Message-ID: References: <1535378254-2593-1-git-send-email-oleg.karfich@wago.com> <54f919ba308c6df58ba98eeb7859ecb878fef4d2.camel@allegion.com> In-Reply-To: <54f919ba308c6df58ba98eeb7859ecb878fef4d2.camel@allegion.com> Content-Language: en-US Content-ID: <8AA26DF51F76BA40945C555C1B4E97D6@WAGO.com> MIME-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH] commands: of_property: do not free data in general To: Florian.Baeuerle@allegion.com, barebox@lists.infradead.org On 27.08.2018 16:57, Baeuerle, Florian wrote: > Hello Oleg, Hi Florian, > > You are right, it works only if the property does not already exist. > In that case we'd probably also need a free() in the else-branch, > where of_new_property is called (otherwise we have a leak), because > that actually copies what is contained in data. You are right, too. Sorry missed this path. V2 is comming soon... > > Sorry for that regression. No problem :-) > Best Regards > Florian Regards Oleg > > Am Montag, den 27.08.2018, 13:57 +0000 schrieb Oleg.Karfich@wago.com: >> With commit 748d1b8 the function do_of_property_set_now is introduced. With this >> commit an unwanted free(data) is added wich leads to that an free on the value >> is done that is previously set to the property. As a consequence the command >> of_dump reads freed data an printis the value as hex. Fix this by moving >> free(data) to the right place like before the above commit. >> >> Signed-off-by: Oleg Karfich >> --- >> commands/of_property.c | 7 ++++--- >> 1 file changed, 4 insertions(+), 3 deletions(-) >> >> diff --git a/commands/of_property.c b/commands/of_property.c >> index 8ce12e5..ea32e51 100644 >> --- a/commands/of_property.c >> +++ b/commands/of_property.c >> @@ -275,10 +275,12 @@ static int do_of_property_set_now(struct device_node *root, const char *path, >> pp->value_const = NULL; >> >> /* limit property data to the actual size */ >> - if (len) >> + if (len) { >> pp->value = xrealloc(data, len); >> - else >> + } else { >> pp->value = NULL; >> + free(data); >> + } >> >> pp->length = len; >> } else { >> @@ -374,7 +376,6 @@ static int do_of_property(int argc, char *argv[]) >> free(data); >> } else { >> ret = do_of_property_set_now(NULL, path, propname, data, len); >> - free(data); >> } >> >> return ret; _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox