Re: [PATCH 2/2] of: base: of_new_property(): use xstrdup() instead of strdup()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 19, 2017 at 01:20:26PM +0300, Antony Pavlov wrote:
> On Mon, 17 Apr 2017 13:42:10 +0300
> Antony Pavlov <antonynpavlov@xxxxxxxxx> wrote:
> 
> > At the moment of_new_property() uses xfunctions for memory
> > allocation so we can use xstrdup() instead of strdup()
> > for code simplification.
> > 
> > A side effect of this commmit is that
> > the of_new_property() function can't return NULL
> > anymore if CONFIG_OFTREE is set.
> 
> If of_new_property() can't return NULL we can skip all numerous
> checks after of_new_property() call. So the checks are actual only
> if CONFIG_OFTREE isn't set.
> 
> Have you any sugestions on removing of_new_property() return value checks?

I'm not sure removing these checks is a good idea. One possible error
return for of_new_property() could be -EEXIST. Right now we do not check
is the property already exists and just create a second property with
the same name. Also if you consider FIT images: These store a whole
kernel in a device tree property, so the xmalloc() we have for
allocating the data for the property might better be malloc() so that
it can fail.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux