Re: REVIEW: Imminent session patch

> believe should be addressed. The problem being that session_register() > function does not work unless the user has register_globals enabled. I think there is a misunderstanding with regard to how sessions interact with global variables. The session extension makes use of globals only, if register_globals is set. Your patch creates references between track and global variables, even if register_globals is not enabled. Prior to 4.3, the session module did not create any references upon session_register() in the register_globals=1 case. This however is necesssary to make get_session_var() reliably work and avoid any lookups in the global sym table. Because this behaviour is new, I don't see how existing applications could break. - Sascha