Skip to content

Commit

Permalink
Merge pull request laruence#51 from wenjun1055/patch-1
Browse files Browse the repository at this point in the history
Thanks for the patch
  • Loading branch information
laruence committed Aug 12, 2013
2 parents f6b910c + 0131e02 commit a836f3a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion configs/yaf_config_simple.c
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@ PHP_METHOD(yaf_config_simple, offsetUnset) {
return;
}

if (Z_TYPE_P(name) != IS_STRING || Z_TYPE_P(name) != IS_STRING) {
if (Z_TYPE_P(name) != IS_STRING || !Z_STRLEN_P(name)) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Expect a string key name");
RETURN_FALSE;
}
Expand Down

0 comments on commit a836f3a

Please sign in to comment.