user/sven/linux.git/drivers/of/resolver.c, branch v4.14.266

of: overlay: validate offset from property fixups

2018-07-03T09:24:52Z

commit 482137bf2aecd887ebfa8756456764a2f6a0e545 upstream. The smatch static checker marks the data in offset as untrusted, leading it to warn: drivers/of/resolver.c:125 update_usages_of_a_phandle_reference() error: buffer underflow 'prop->value' 's32min-s32max' Add check to verify that offset is within the property data. Reported-by: Dan Carpenter Signed-off-by: Frank Rowand Cc: Signed-off-by: Rob Herring Signed-off-by: Greg Kroah-Hartman

of: find_node_by_full_name rewrite to compare each level

2017-06-22T17:38:29Z

find_node_by_full_name() does the same thing as of_find_node_by_path(), but takes no locks and doesn't work on aliases. Refactor of_find_node_opts_by_path() into __of_find_node_by_full_path() and replace find_node_by_full_name() with it. Signed-off-by: Rob Herring

of: use kbasename instead of open coding

2017-06-22T17:37:46Z

Several places in DT code open code the equivalent of kbasename. Replace them. The behavior for root nodes in node_name_cmp will be slightly different. Instead of comparing "/", "" will be compared. The comparison will be the same. Reviewed-by: Andy Shevchenko Signed-off-by: Rob Herring

of: fix sparse warnings in fdt, irq, reserved mem, and resolver code

2017-05-04T19:01:13Z

sparse generates the following warnings in drivers/of/: ../drivers/of/fdt.c:63:36: warning: cast to restricted __be32 ../drivers/of/fdt.c:68:33: warning: cast to restricted __be32 ../drivers/of/irq.c:105:88: warning: incorrect type in initializer (different base types) ../drivers/of/irq.c:105:88: expected restricted __be32 ../drivers/of/irq.c:105:88: got int ../drivers/of/irq.c:526:35: warning: incorrect type in assignment (different modifiers) ../drivers/of/irq.c:526:35: expected int ( *const [usertype] irq_init_cb )( ... ) ../drivers/of/irq.c:526:35: got void const *const data ../drivers/of/of_reserved_mem.c:200:50: warning: incorrect type in initializer (different modifiers) ../drivers/of/of_reserved_mem.c:200:50: expected int ( *[usertype] initfn )( ... ) ../drivers/of/of_reserved_mem.c:200:50: got void const *const data ../drivers/of/resolver.c:95:42: warning: incorrect type in assignment (different base types) ../drivers/of/resolver.c:95:42: expected unsigned int [unsigned] [usertype] ../drivers/of/resolver.c:95:42: got restricted __be32 [usertype] All these are harmless type mismatches fixed by adjusting the types. Signed-off-by: Rob Herring

of: drop duplicate headers

2017-01-04T20:01:28Z

Drop duplicate headers string.h and of_platform.h. Signed-off-by: Geliang Tang Signed-off-by: Rob Herring

of: Fix issue where code would fall through to error case.

2016-12-09T16:33:03Z

No longer fall through into the error case that prints out an error if no error (err = 0) occurred. Fixes d9181b20a83(of: Add back an error message, restructured) Signed-off-by: Moritz Fischer Reviewed-by: Frank Rowand Signed-off-by: Rob Herring

of: Remove unused variable overlay_symbols

2016-11-15T15:17:18Z

Remove unused pointer to node "__symbols__". Signed-off-by: Frank Rowand Signed-off-by: Rob Herring

of: Move setting of pointer to beside test for non-null

2016-11-15T15:17:17Z

Value of pointer was calculated in an earlier block than where it was used. Move it down into the block where it is used, immediately before where is is checked to be valid. Signed-off-by: Frank Rowand Signed-off-by: Rob Herring

of: Add back an error message, restructured

2016-11-15T15:17:17Z

Add a single pr_err() to cover a range of errors that were reported by several pr_err() that were removed earlier in this series. Signed-off-by: Frank Rowand Signed-off-by: Rob Herring

of: Update comments to reflect changes and increase clarity

2016-11-15T15:17:16Z

Update comments to better explain what functions are doing. Signed-off-by: Frank Rowand Signed-off-by: Rob Herring