From 5b695f393e6777aad4618d55f2b507e57bb4dc4b Mon Sep 17 00:00:00 2001 From: Doug Zongker Date: Wed, 24 Feb 2010 15:03:47 -0800 Subject: [PATCH] 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 --- edify/expr.c | 1 + 1 file changed, 1 insertion(+) 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);