From 1ac2cb974142132d9bfd988e6a8702d7a66958e0 Mon Sep 17 00:00:00 2001 From: Patrice Chotard Date: Mon, 25 Nov 2019 09:07:36 +0100 Subject: [PATCH] cmd: Migrate from_env() from pxe.c to nvedit.c Migrate from_env() from pxe.c to nvedit.c as it's not pxe specific. Signed-off-by: Patrice Chotard --- cmd/nvedit.c | 17 +++++++++++++++++ cmd/pxe.c | 17 ----------------- include/env.h | 10 ++++++++++ 3 files changed, 27 insertions(+), 17 deletions(-) diff --git a/cmd/nvedit.c b/cmd/nvedit.c index b5da375913..3420e0b985 100644 --- a/cmd/nvedit.c +++ b/cmd/nvedit.c @@ -681,6 +681,23 @@ char *env_get(const char *name) return NULL; } +/* + * Like env_get, but prints an error if envvar isn't defined in the + * environment. It always returns what env_get does, so it can be used in + * place of env_get without changing error handling otherwise. + */ +char *from_env(const char *envvar) +{ + char *ret; + + ret = env_get(envvar); + + if (!ret) + printf("missing environment variable: %s\n", envvar); + + return ret; +} + /* * Look up variable from environment for restricted C runtime env. */ diff --git a/cmd/pxe.c b/cmd/pxe.c index 768e50aba6..ed553953b3 100644 --- a/cmd/pxe.c +++ b/cmd/pxe.c @@ -37,23 +37,6 @@ const char *pxe_default_paths[] = { static bool is_pxe; -/* - * Like env_get, but prints an error if envvar isn't defined in the - * environment. It always returns what env_get does, so it can be used in - * place of env_get without changing error handling otherwise. - */ -static char *from_env(const char *envvar) -{ - char *ret; - - ret = env_get(envvar); - - if (!ret) - printf("missing environment variable: %s\n", envvar); - - return ret; -} - #ifdef CONFIG_CMD_NET /* * Convert an ethaddr from the environment to the format used by pxelinux diff --git a/include/env.h b/include/env.h index b72239f6a5..d6c2d751d6 100644 --- a/include/env.h +++ b/include/env.h @@ -113,6 +113,16 @@ int env_match(unsigned char *name, int index); */ char *env_get(const char *varname); +/* + * Like env_get, but prints an error if envvar isn't defined in the + * environment. It always returns what env_get does, so it can be used in + * place of env_get without changing error handling otherwise. + * + * @varname: Variable to look up + * @return value of variable, or NULL if not found + */ +char *from_env(const char *envvar); + /** * env_get_f() - Look up the value of an environment variable (early) * -- GitLab