Skip to content

Conversation

@reeze
Copy link
Contributor

@reeze reeze commented Apr 29, 2012

Hi,
in cli sapi SG(request_info).path_translated is relative path. when chdir it will point to non-exists file.
in fastcgi and fpm they all point to absolute path. so I made this push request to fix this.

this bug affect get_current_user(), getmyinode() and getlastmod().

thanks.

@laruence
Copy link
Member

Hi, you should not call to tsrm_realpath even the scriptname is "-", just some lines above it there is the obvious codes to set filename to '-', didn't you see it?

Any, I Committed a little improved patch. thanks

@php-pulls
Copy link

Comment on behalf of laruence at php.net:

Closed

@php-pulls php-pulls closed this Apr 30, 2012
@reeze
Copy link
Contributor Author

reeze commented Apr 30, 2012

oh, that's the truth. I didn't noticed that.

@laruence
Copy link
Member

On Mon, Apr 30, 2012 at 12:38 PM, reeze
reply@reply.github.com
wrote:

oh, that's the truth. I didn't noticed that.
that's okey, but when we trying to fix sth, we should be careful and
make sure you really understood the depended codes.

anyway, thanks for spotting this. committed here:
bae56a8

thanks


Reply to this email directly or view it on GitHub:
#69 (comment)

惠新宸        laruence
Senior PHP Engineer
http://www.laruence.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants