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

HHVM Crashes on file change #3482

Closed
infusion opened this issue Aug 15, 2014 · 3 comments
Closed

HHVM Crashes on file change #3482

infusion opened this issue Aug 15, 2014 · 3 comments

Comments

@infusion
Copy link

Hi,

HHVM segfaults quite often, when I push a new code base with rsync into the docroot. I'm not sure if it's a problem of the stat cache/inotify implementation.

How to reproduce:

  • Change some code
  • rsync into the docroot
  • reload page
    .....

[pid 12935] lseek(26, 1728512, SEEK_SET) = 1728512
[pid 12935] read(26, "setlogin is not implemented and "..., 4096) = 4096
[pid 12935] lseek(26, 1732608, SEEK_SET) = 1732608
[pid 12935] read(26, "y\2\0\0\1\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\340Z\32\0\0\0\0\0"..., 4096) = 1472
[pid 12935] lseek(26, 1728512, SEEK_SET) = 1728512
[pid 12935] read(26, "setlogin is not implemented and "..., 4096) = 4096
[pid 12935] read(26, "y\2\0\0\1\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\340Z\32\0\0\0\0\0"..., 4096) = 1472
[pid 12935] lseek(26, 0, SEEK_SET) = 0
[pid 12935] read(26, "\177ELF\2\1\1\3\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0P\34\2\0\0\0\0\0"..., 4096) = 4096
[pid 12935] lseek(26, 1728512, SEEK_SET) = 1728512
[pid 12935] read(26, "setlogin is not implemented and "..., 4096) = 4096
[pid 12935] lseek(26, 0, SEEK_SET) = 0
[pid 12935] read(26, "\177ELF\2\1\1\3\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0P\34\2\0\0\0\0\0"..., 4096) = 4096
[pid 12935] --- SIGSEGV (Segmentation fault) @ 0 (0) ---
Process 12935 detached
[pid 12934] +++ killed by SIGSEGV +++
+++ killed by SIGSEGV +++

on strace: strace -p 12928 -f

@fcorriga
Copy link

confirmed (3.1 and 3.2). it crashes if you reload the page WHILE the file it's being synched.
it happens thru ftp/sftp too.

@ptarjan ptarjan added the crash label Aug 20, 2014
@fredemmott
Copy link
Contributor

Fix up internally (D1637887) - should hopefully land soon.

@fredemmott
Copy link
Contributor

Including modified files - same basic issue as #1447

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

No branches or pull requests

4 participants