diff options
| author | Doug Zongker <dougz@android.com> | 2010-02-24 15:03:47 -0800 |
|---|---|---|
| committer | Doug Zongker <dougz@android.com> | 2010-02-24 15:05:07 -0800 |
| commit | 5b695f393e6777aad4618d55f2b507e57bb4dc4b (patch) | |
| tree | a8e3ef7c71386ac617b48d9f18ca19edfb7e33a5 /edify/expr.c | |
| parent | c4351c791052ad529a4e83c600b1aa6e6420ea86 (diff) | |
| download | bootable_recovery-5b695f393e6777aad4618d55f2b507e57bb4dc4b.zip bootable_recovery-5b695f393e6777aad4618d55f2b507e57bb4dc4b.tar.gz bootable_recovery-5b695f393e6777aad4618d55f2b507e57bb4dc4b.tar.bz2 | |
make StringValue wrapper okay to call on NULL
The docs say "don't do this", but it's trivial to make safe. Make
StringValue(NULL) return NULL instead of crashing.
Change-Id: I2221bcb4c98d8adb4e25c764d7bdcfa787822bcf
Diffstat (limited to 'edify/expr.c')
| -rw-r--r-- | edify/expr.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/edify/expr.c b/edify/expr.c index 7a5b2fb..3600075 100644 --- a/edify/expr.c +++ b/edify/expr.c @@ -50,6 +50,7 @@ Value* EvaluateValue(State* state, Expr* expr) { } Value* StringValue(char* str) { + if (str == NULL) return NULL; Value* v = malloc(sizeof(Value)); v->type = VAL_STRING; v->size = strlen(str); |