summaryrefslogtreecommitdiff
path: root/common/env_fat.c
diff options
context:
space:
mode:
authorTom Rini <trini@ti.com>2013-04-05 14:55:21 -0400
committerTom Rini <trini@ti.com>2013-04-05 14:55:21 -0400
commitcd0f4fa1ca2901312ae78bc27d4edc8286fcbf1d (patch)
tree244e08cd2c8702c9c6c5854eece41377716dbd31 /common/env_fat.c
parentbc5fd908d976cfd898e8cbb591e7220ddc8a684a (diff)
downloadu-boot-imx-cd0f4fa1ca2901312ae78bc27d4edc8286fcbf1d.zip
u-boot-imx-cd0f4fa1ca2901312ae78bc27d4edc8286fcbf1d.tar.gz
u-boot-imx-cd0f4fa1ca2901312ae78bc27d4edc8286fcbf1d.tar.bz2
Revert "env: fix potential stack overflow in environment functions"
Wolfgang requested this be reverted and Rob agreed after further discussion. This was a symptom of a larger problem we need to deal with. This reverts commit 60d7d5a63189c9f77a190c9965861dc15482c2d0. Signed-off-by: Tom Rini <trini@ti.com>
Diffstat (limited to 'common/env_fat.c')
-rw-r--r--common/env_fat.c11
1 files changed, 5 insertions, 6 deletions
diff --git a/common/env_fat.c b/common/env_fat.c
index dd7139d..c0f18ab 100644
--- a/common/env_fat.c
+++ b/common/env_fat.c
@@ -37,7 +37,6 @@
char *env_name_spec = "FAT";
env_t *env_ptr;
-static char env_buf[CONFIG_ENV_SIZE];
DECLARE_GLOBAL_DATA_PTR;
@@ -53,7 +52,7 @@ int env_init(void)
#ifdef CONFIG_CMD_SAVEENV
int saveenv(void)
{
- env_t *env_new = env_buf;
+ env_t env_new;
ssize_t len;
char *res;
block_dev_desc_t *dev_desc = NULL;
@@ -61,7 +60,7 @@ int saveenv(void)
int part = FAT_ENV_PART;
int err;
- res = (char *)env_new->data;
+ res = (char *)&env_new.data;
len = hexport_r(&env_htab, '\0', 0, &res, ENV_SIZE, 0, NULL);
if (len < 0) {
error("Cannot export environment: errno = %d\n", errno);
@@ -96,8 +95,8 @@ int saveenv(void)
return 1;
}
- env_new->crc = crc32(0, env_new->data, ENV_SIZE);
- err = file_fat_write(FAT_ENV_FILE, (void *)env_new, sizeof(env_t));
+ env_new.crc = crc32(0, env_new.data, ENV_SIZE);
+ err = file_fat_write(FAT_ENV_FILE, (void *)&env_new, sizeof(env_t));
if (err == -1) {
printf("\n** Unable to write \"%s\" from %s%d:%d **\n",
FAT_ENV_FILE, FAT_ENV_INTERFACE, dev, part);
@@ -111,7 +110,7 @@ int saveenv(void)
void env_relocate_spec(void)
{
- char *buf = env_buf;
+ char buf[CONFIG_ENV_SIZE];
block_dev_desc_t *dev_desc = NULL;
int dev = FAT_ENV_DEVICE;
int part = FAT_ENV_PART;