Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix read of 8192 bytes failed with errno=21 Is a directory #7

Closed
weierophinney opened this issue Dec 31, 2019 · 0 comments · Fixed by #84
Closed

fix read of 8192 bytes failed with errno=21 Is a directory #7

weierophinney opened this issue Dec 31, 2019 · 0 comments · Fixed by #84
Labels
Bug Something isn't working
Milestone
2.10.1

Comments

@weierophinney
Copy link
Member

With 7.4.0RC3

There was 1 error:
1) ZendTest\Mail\Storage\MboxFolderTest::testChangeFolderUnselectable
fgets(): read of 8192 bytes failed with errno=21 Is a directory
/builddir/build/BUILDROOT/php-zendframework-zend-mail-2.10.0-5.fc32.noarch/usr/share/php/Zend/Mail/Storage/Mbox.php:232
/builddir/build/BUILDROOT/php-zendframework-zend-mail-2.10.0-5.fc32.noarch/usr/share/php/Zend/Mail/Storage/Mbox.php:268
/builddir/build/BUILDROOT/php-zendframework-zend-mail-2.10.0-5.fc32.noarch/usr/share/php/Zend/Mail/Storage/Folder/Mbox.php:164
/builddir/build/BUILD/zend-mail-d7beb63d5f7144a21ac100072c453e63860cdab8/test/Storage/MboxFolderTest.php:143

This is a trivial fix which "also" mute E_NOTICE raised by fgets

Originally posted by @remicollet at zendframework/zend-mail#244
@weierophinney weierophinney mentioned this issue Dec 31, 2019
Open
@michalbundyra michalbundyra mentioned this issue Jan 15, 2020
Open
@michalbundyra michalbundyra mentioned this issue Mar 31, 2020
Merged
@michalbundyra michalbundyra added Bug Something isn't working and removed Enhancement labels Mar 31, 2020
@michalbundyra michalbundyra added this to the 2.10.1 milestone Mar 31, 2020
@Ocramius Ocramius mentioned this issue Feb 5, 2021
Merged
artemii-karkusha pushed a commit to artemii-karkusha/laminas-mail that referenced this issue May 24, 2023
@michalbundyra
Hotfix: PHP 7.4 compatibility
c1b09a5 
Throw exception earlier if provided filename is a directory.
We except the filename to be mbox file.

Fixes laminas#7

Signed-off-by: Michał Bundyra <contact@webimpress.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Something isn't working
Projects
None yet
Milestone
2.10.1
Development

Successfully merging a pull request may close this issue.

Hotfix: PHP 7.4 compatibility
2 participants
@weierophinney @michalbundyra