[PATCH RFC] common: env: handle var="" not as var=NULL

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

 



This patch changes the behaviour which I think is used everywhere in
barebox code. Currently var="" and var=NULL will end in the same
behaviour.

New behaviour is:

setenv(name, "")	-> sets name to ""
\-> getenv(name)	-> ""

setenv(name, NULL)	-> deletes env
\-> getenv(name)	-> NULL

It seems that barebox code already wants such behaviour, by run

$ grep -r "\"\"" | grep setenv

it finds most all related code sections which use "".

I detected the issue while commenting #ip="foo" in my env. The code
part:

net/ifup.c:	setenv("ip", "");

Sets at first "ip" to "" but will end in the old behaviour as "delete".
The getenv("ip") in ifup.c will not check on the NULL value and has a
general handling for unknown ip env.

Note:
Also check commit e7ed69d5 ("setenv: be robust against setenv(<var>,
NULL") which says that hush will handle "" and NULL as the same.

Another check would be:

$ grep -r "NULL" | grep setenv

which should show the behaviour in case of deleting env.

Signed-off-by: Alexander Aring <aar@xxxxxxxxxxxxxx>
---
I test the ip env stuff only that barebox will not crash anymore but I
think it will break some other setenv/getenv behaviour.

The question is "Shall we keep that "" and NULL ends in same behaviour"
and fix the code which use it wrong, e.g. setenv("ip", ""); getenv("ip").

 common/env.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/common/env.c b/common/env.c
index 6f736d5..4d8d446 100644
--- a/common/env.c
+++ b/common/env.c
@@ -210,10 +210,6 @@ int setenv(const char *_name, const char *value)
 	int ret = 0;
 	struct list_head *list;
 
-	if (value && !*value)
-		value = NULL;
-
-
 	if ((par = strchr(name, '.'))) {
 		struct device_d *dev;
 
-- 
2.8.3


_______________________________________________
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