[BUG] Phalcon crashes when NuSphere debugger is enabled - *solution included*

[temuri416]

@phalcon @niden

Guys, after testing various commits I've found a problem code that is the root cause of the following issues:
#2356 #2118 #2104

The following two lines should be removed:

https://github.com/phalcon/cphalcon/blob/master/ext/session/adapter.c#L399
https://github.com/phalcon/cphalcon/blob/master/ext/session/adapter.c#L410

It was a part of commit ID 3aecf06 and caused all of my problems with debugging Phalcon-based application.

I've tested it several times - latest 1.3.2 without those two lines is rock-solid.

Thanks!!

---

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

[niden]

Thanks @temuri416

From what I see in the commits the variable is used to detect whether DBG has been installed here:

cphalcon/ext/phalcon.c

Line 223 in 27facc2

nusphere_dbg_present = (zend_get_extension("DBG") != NULL);

I haven't used nusphere before so I cannot comment on that, but doesn't DBG report that it is installed? If so the if statement seems legit.

[temuri416]

@niden

That variable correctly reflects presence of DBG. All other checks work as they should. It's just those two lines that mess it all up.

Can they be removed?

Thanks!

[niden]

@temuri416 #2454

[niden]

@temuri416 Please run this through your system and let us know if that one resolved it.

[temuri416]

Awesome, thanks! I'll test it ASAP.