Please forgive me if I'm wrong, but I happened to look at the changes for [262] (and [261]) and noticed the following:
if (eaccelerator_mm_instance == NULL && eaccelerator_list_keys(return_value TSRMLS_CC)) {
return;
} else {
RETURN_NULL ();
}
Shouldn't that be...
if (eaccelerator_mm_instance != NULL && eaccelerator_list_keys(return_value TSRMLS_CC)) {
return;
} else {
RETURN_NULL ();
}
From my reading of the code, the current state means eaccelerator_list_keys() will most likely not work, and if eaccelerator_mm_instance is null, bad mojo could happen?
-[Unknown]
