Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1629205 > unrolled thread

[PATCH 3/3] of: detect invalid phandle in overlay

Started byfrowand.list@gmail.com
First post2017-04-24 07:30 +0200
Last post2017-04-24 07:30 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH 3/3] of: detect invalid phandle in overlay frowand.list@gmail.com - 2017-04-24 07:30 +0200

#1629205 — [PATCH 3/3] of: detect invalid phandle in overlay

Fromfrowand.list@gmail.com
Date2017-04-24 07:30 +0200
Subject[PATCH 3/3] of: detect invalid phandle in overlay
Message-ID<tzEJr-2O1-9@gated-at.bofh.it>
From: Frank Rowand <frank.rowand@sony.com>

Overlays are not allowed to modify phandle values of previously existing
nodes because there is no information available to allow fixup up of
properties that use the previously existing phandle.

Signed-off-by: Frank Rowand <frank.rowand@sony.com>
---
 drivers/of/overlay.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/of/overlay.c b/drivers/of/overlay.c
index 7827786718d8..c0e4ee1cd1ba 100644
--- a/drivers/of/overlay.c
+++ b/drivers/of/overlay.c
@@ -132,6 +132,10 @@ static int of_overlay_apply_single_device_node(struct of_overlay *ov,
 	/* NOTE: Multiple mods of created nodes not supported */
 	tchild = of_get_child_by_name(target, cname);
 	if (tchild != NULL) {
+		/* new overlay phandle value conflicts with existing value */
+		if (child->phandle)
+			return -EINVAL;
+
 		/* apply overlay recursively */
 		ret = of_overlay_apply_one(ov, tchild, child);
 		of_node_put(tchild);
-- 
Frank Rowand <frank.rowand@sony.com>

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web