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.

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

tail: Do not trust st_size if it equals zero #990

Closed
wants to merge 1 commit into from

Conversation

ricardobranco777
Copy link
Contributor

@ricardobranco777 ricardobranco777 commented Jan 3, 2024

Fix for tail not being able to operate on files residing in pseudo-filesystems that advertize a zero size value.

To reproduce: tail -1 /compat/linux/proc/mounts

Bug: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=276107

@delphij delphij self-requested a review January 4, 2024 08:48
@delphij delphij self-assigned this Jan 4, 2024
@delphij
Copy link
Member

delphij commented Jan 4, 2024

LGTM; pushed to main.

@delphij delphij closed this Jan 4, 2024
@emaste emaste added the merged label Jan 4, 2024
@emaste
Copy link
Member

emaste commented Jan 4, 2024

1fb3cae

@emaste emaste reopened this Jan 4, 2024
@emaste emaste closed this Jan 4, 2024
@emaste emaste reopened this Jan 4, 2024
@emaste emaste closed this Jan 4, 2024
@emaste
Copy link
Member

emaste commented Jan 4, 2024

(oops, clicked the wrong button and reopened by accident)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants