Re: [U-Boot] [PATCH 10/12] env: acl: Add environment variable access control list

2012-09-14 Thread Wolfgang Denk
Dear Joe Hershberger, In message you wrote: > > I guess you didn't see this the last time I sent it to you off list... I didn't receive any such reply, sorry. Do you have the message ID and/or exact timestanmp of your message? > > - It appears you will store all access related information i

Re: [U-Boot] [PATCH 10/12] env: acl: Add environment variable access control list

2012-09-13 Thread Joe Hershberger
Hi Wolfgang, I guess you didn't see this the last time I sent it to you off list... On Thu, Sep 13, 2012 at 3:13 PM, Wolfgang Denk wrote: > Dear Joe Hershberger, > > In message <1345236586-19076-11-git-send-email-joe.hershber...@ni.com> you > wrote: >> Currently just validates variable types as

Re: [U-Boot] [PATCH 10/12] env: acl: Add environment variable access control list

2012-09-13 Thread Wolfgang Denk
Dear Joe Hershberger, In message <1345236586-19076-11-git-send-email-joe.hershber...@ni.com> you wrote: > Currently just validates variable types as decimal, hexidecimal, > boolean, ip address, and mac address. Call > env_acl_validate_setenv_params() from setenv() in both cmd_nvedit.c > and fw_e

Re: [U-Boot] [PATCH 10/12] env: acl: Add environment variable access control list

2012-08-22 Thread Mike Frysinger
On Friday 17 August 2012 16:49:44 Joe Hershberger wrote: > --- a/common/cmd_nvedit.c > +++ b/common/cmd_nvedit.c > > +#if defined(CONFIG_ENV_ACL) > +#include > +#endif the header should not need protection just to be included > +#ifdef CONFIG_ENV_ACL > + if (env_acl_validate_env_set_params(a

[U-Boot] [PATCH 10/12] env: acl: Add environment variable access control list

2012-08-17 Thread Joe Hershberger
Currently just validates variable types as decimal, hexidecimal, boolean, ip address, and mac address. Call env_acl_validate_setenv_params() from setenv() in both cmd_nvedit.c and fw_env.c. If the entry is not found in the env ACL, then look in the static one. This allows the env to override the