9
0
Fork 0
barebox/fs
Sascha Hauer 1a9e93cc81 drivers/base: fix corrupt device tree
dev_add_child is a very unsafe function. If called multiple times
it allows setting the same device to different parents thus corrupting
the siblings list. This happens regularly since:

| commit c2e568d19c
| Author: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
| Date:   Sat Nov 3 16:11:05 2012 +0100
|
|    bus: add bus device
|
|    automatically add it as parent of any bus device if none already specified
|
|    we have now a nice output per bus

If for example a FATfs is mounted this nice output per bus often ends with:

>     `---- fat0
>     `---- 0
>          `---- 0x86f0000087020031-0x86f000410df27124: /dev/<NULL>
>          `---- sram00
>               `---- 0x00000000-0xffffffffffffffff: /dev/<NULL>
>               `---- 0x00000000-0xffffffffffffffff: /dev/<NULL>
>               unable to handle NULL pointer dereference at address 0x0000000c
> pc : [<87f08a20>]    lr : [<87f08a04>]
> sp : 86eff8c0  ip : 87f3fbde  fp : ffffffff
> r10: ffffffff  r9 : 00000000  r8 : 00000003
> r7 : 86f075b8  r6 : 00000002  r5 : ffffffec  r4 : 86f07544
> r3 : 00000000  r2 : 43f900b4  r1 : 00000020  r0 : 00000005
> Flags: Nzcv  IRQs off  FIQs off  Mode SVC_32
> [<87f08a20>] (do_devinfo_subtree+0x90/0x130) from [<87f08a90>] (do_devinfo_subtree+0x100/0x130)
>
> [<87f3e070>] (unwind_backtrace+0x0/0x90) from [<87f28514>] (panic+0x28/0x3c)
> [<87f28514>] (panic+0x28/0x3c) from [<87f3e4b8>] (do_exception+0x10/0x14)
> [<87f3e4b8>] (do_exception+0x10/0x14) from [<87f3e544>] (do_data_abort+0x2c/0x38)
> [<87f3e544>] (do_data_abort+0x2c/0x38) from [<87f3e268>] (data_abort+0x48/0x60)

This patch fixes this by adding a device to its parents children list in
register_device so that dev_add_child is no longer needed. This function
is removed from the tree. Now callers of register_device have to clearly
set the parent *before* registering a device.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Reported-by: Jan Lübbe <jlu@pengutronix.de>
2012-12-12 15:04:27 +01:00
..
cramfs use loff_t for file offsets 2012-06-30 16:53:24 +02:00
ext4 fs: implement initial ext4 support from U-Boot 2012-12-03 11:23:25 +01:00
fat Merge branch 'for-next/remove-fsf-address' 2012-10-03 21:12:48 +02:00
Kconfig Merge branch 'kconfig' 2012-12-08 12:22:21 +01:00
Makefile fs: implement initial ext4 support from U-Boot 2012-12-03 11:23:25 +01:00
devfs-core.c devfs-core: add writesize in mtd partition ioctl 2012-10-23 08:25:31 +02:00
devfs.c Treewide: remove address of the Free Software Foundation 2012-09-17 10:57:41 +02:00
fs.c drivers/base: fix corrupt device tree 2012-12-12 15:04:27 +01:00
nfs.c Treewide: remove address of the Free Software Foundation 2012-09-17 10:57:41 +02:00
omap4_usbbootfs.c omap4: add filesystem support over usb boot 2012-11-16 15:32:31 +01:00
ramfs.c Treewide: remove address of the Free Software Foundation 2012-09-17 10:57:41 +02:00
tftp.c mem: add the swab (swap bytes) option to memory_display() 2012-11-26 11:14:40 +01:00