Skip to content
This repository has been archived by the owner on Apr 1, 2023. It is now read-only.

Fix bug: tmp_name check #278

Closed
wants to merge 1 commit into from
Closed

Fix bug: tmp_name check #278

wants to merge 1 commit into from

Conversation

mpyw
Copy link

@mpyw mpyw commented Aug 6, 2016

The form names containing tmp_name may cause errors on fopen() call.

<form method="post" action="" enctype="multipart/form-data">
    <input type="submit">
    <input type="file" name="file[x][tmp_name][y]">
</form>

So we should check if tmp_name is not array using !is_array().

@fabpot
Copy link
Member

fabpot commented Dec 6, 2019

Closing as Goutte 4 has just been released and it does not depend on Guzzle anymore.

@fabpot fabpot closed this Dec 6, 2019
@mpyw mpyw deleted the fix-tmp_name-check branch December 6, 2019 15:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants