search for: eintr

http://bugzilla.mindrot.org/show_bug.cgi?id=52 dtucker at zip.com.au changed: What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |1274 nThis| | ------- You are receiving this mail because: ------- You are the assignee for

https://bugzilla.mindrot.org/show_bug.cgi?id=3071 Bug ID: 3071 Summary: unhandled EINTR while connecting causes ssh to exit prematurely Product: Portable OpenSSH Version: 8.0p1 Hardware: amd64 OS: Linux Status: NEW Severity: trivial Priority: P5 Component: ssh Assigne...

...rate (in /usr/lib/x86_64-linux-gnu/libsmbconf.so.0) > ==2796353== by 0x716BD6E: ??? (in /usr/lib/x86_64-linux-gnu/libsmbconf.so.0) > ==2796353== by 0xAE6692F: ??? (in /usr/lib/x86_64-linux-gnu/samba/libdbwrap.so.0) > > Ifound the sendmsg is always failed because erron=EINTR, but smbd also need to malloc for the new msgs, so the res of the smbd grows up quickly. > > I add some code in unix_dgram_send_job, just send 10 times if sendmsg faild with EINTR, the res will not grows up anymore. > Another, keep the max queue length to 100 also work well. > > I...

As Eric Blake noted in: https://www.redhat.com/archives/libguestfs/2016-April/msg00154.html libguestfs doesn't correctly handle the case where waitpid receives a SIGCHLD signal and the main program has registered a non-restartable signal handler. In this case waitpid would return -EINTR and we would print an error, but actually we should retry this case. This adds two new internal functions, one for handling waitpid and printing (real) errors from it, the other for just waiting for a process where we don't particularly care about errors. Rich.

...lude <sys/wait.h> #include "ignore-value.h" @@ -91,18 +90,13 @@ guestfs_int_getumask (guestfs_h *g) if (read (fd[0], &mask, sizeof mask) != sizeof mask) { perrorf (g, "read"); close (fd[0]); - while (waitpid (pid, NULL, 0) == -1 && errno == EINTR) - ; + guestfs_int_waitpid_noerror (pid); return -1; } close (fd[0]); - again: - if (waitpid (pid, &status, 0) == -1) { - if (errno == EINTR) goto again; - perrorf (g, "waitpid"); + if (guestfs_int_waitpid (g, pid, &status, "umask") == -1)...

Hi I have a process which reads a socket using recvfrom. After sometime, for no apparent reason, recvfrom exits with an EINTR causing the sender to barf too. Basically the socket on which these processes were communicating no longer exists. On writing a dtrace, I could determine that a signal was received at that socket causing the EINTR. I do not know of way to see who sent the signal (which process) and what the signal...

...>> Oops - this strands a child process. You have to reap the child, even > >> if the read() failed. > > > > Bleah that was stupid. Try attached version - the only difference is > > I added a waitpid call to the error path above. > > But without looping on EINTR... Thanks - I pushed it with your suggested loop added. BTW we don't loop in waitpid/EINTR anywhere else in libguestfs, but I guess this is something we should be doing. Can you tell us why it's necessary? Rich. > > + > + /* Read the umask. */ > + if (read (fd[0], &m...

...left "hanging" when you normally exit from a ssh shell (for example by logging out from the remote host via "exit" or "logout"). The problem seems to be that sshd (and some other parts of OpenSSH) doesn't check the return code and errno from waitpid() for errno == EINTR and thus fails to collect the started subprocess correctly. This problem occurs atleast on Solaris 2 and UNICOS systems but it should probably exist on all other SysV based systems... (It doesn't occur all the time so it can be quite hard to test for). The other patch adds a "-I" com...

On Thu, Apr 14, 2016 at 07:38:23AM -0600, Eric Blake wrote: > > + /* Read the umask. */ > > + if (read (fd[0], &mask, sizeof mask) != sizeof mask) { > > + perrorf (g, "read"); > > + close (fd[0]); > > + return -1; > > Oops - this strands a child process. You have to reap the child, even > if the read() failed. Bleah that was

dovecot 0.99 still (RHEL4), I'm sorry to say. The config is for mbox's to live on the user's home disk. If the user tries to make an imap connection when his home disk is unmounted, various permission denied messages are logged, as expected, but then the imap process seems to hang around instead of aborting. The result is that eventually the imap process limit is reached, and even

Call reservation_object_* directly instead of using ttm_bo_{reserve,unreserve}. v4: check for EINTR only. v3: check for EINTR too. Signed-off-by: Gerd Hoffmann <kraxel at redhat.com> Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch> --- drivers/gpu/drm/virtio/virtgpu_drv.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_d...

Call reservation_object_* directly instead of using ttm_bo_{reserve,unreserve}. v4: check for EINTR only. v3: check for EINTR too. Signed-off-by: Gerd Hoffmann <kraxel at redhat.com> Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch> --- drivers/gpu/drm/virtio/virtgpu_drv.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_d...

Oeps, sorry wrong one. There where multiple fixes. https://wiki.samba.org/index.php/Samba_4.4_Features_added/changed BUG #12377: vfs_glusterfs: Fix a memory leak in connect path. https://wiki.samba.org/index.php/Samba_4.5_Features_added/changed BUG #12485: ctdbd_conn: Fix a resource leak. https://wiki.samba.org/index.php/Samba_4.6_Features_added/changed ## BUG #12624: lib/pthreadpool: Fix

Call reservation_object_* directly instead of using ttm_bo_{reserve,unreserve}. v3: check for EINTR too. Signed-off-by: Gerd Hoffmann <kraxel at redhat.com> Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch> --- drivers/gpu/drm/virtio/virtgpu_drv.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.h b/drivers/gpu/drm/vir...

...the meaning of this. Sorry. > > > > > > > > > > Thanks > > > > > > > > Standard way to handle things that can timeout and where userspace > > > > did not supply the time is to block until an interrupt > > > > then return EINTR. > > > > > > Well this seems to be a huge change, ioctl(2) doesn't say it can > > > return EINTR now. > > > > the one on fedora 37 does not but it says: > > No single standard. Arguments, returns, and semantics of ioctl() vary according to th...

The 4.6p1 sshd is logging this error during remote commands or file transfers: error: channel 0: chan_read_failed for istate 3 Platform is Solaris 8, 4.6p1 + OpenSSL 0.9.8d. The commands and transfers work correctly, so the error message appears to be spurious. The error message does not appear when processing logins. Otherwise 4.6p1 is running without any apparent problems. This error

...left "hanging" when you normally exit from a ssh shell (for example by logging out from the remote host via "exit" or "logout"). The problem seems to be that sshd (and some other parts of OpenSSH) doesn't check the return code and errno from waitpid() for errno == EINTR and thus fails to collect the started subprocess correctly. This problem occurs atleast on Solaris 2 and UNICOS systems but it should probably exist on all other SysV based systems... (It doesn't occur all the time so it can be quite hard to test for). The other patch adds a "-I" com...

...le timeouts starts now immediately after authorization has been successfull, not when sshd first enters wait_until_can_do_something() - IMHO, the patch is much cleaner. max_time_seconds variable has been removed. - It fixes two places, where select() might have been called after EAGAIN or EINTR, but the fd_sets would not have been reinitialized. - If select returns error other than EAGAIN or EINTR, it exits with fatal(). Before the error would either be ignored or just logged, but the program would have continued like nothing would have happened. The manual says, that after select(...

...ill effect? +289 +290 /* Read and buffer any available stdout data from the program. */ +291 if (!fdout_eof && FD_ISSET(fdout, readset)) { +292 len = read(fdout, buf, sizeof(buf)); +293 if (len < 0 && (errno == EINTR || errno == EAGAIN)) { +294 /* do nothing */ +295 } else if (len <= 0) { +296 fdout_eof = 1; +297 } else { +298 buffer_append(&stdout_buffer, buf, len); +299...

..., &rfds); +do_select_again: ret = select(qmp->qmp_fd + 1, &rfds, NULL, NULL, &timeout); if (ret == 0) { LIBXL__LOG(qmp->ctx, LIBXL__LOG_ERROR, "timeout"); return -1; } else if (ret < 0) { + if (errno == EINTR) + goto do_select_again; LIBXL__LOG_ERRNO(qmp->ctx, LIBXL__LOG_ERROR, "Select error"); return -1; } rd = read(qmp->qmp_fd, qmp->buffer, QMP_RECEIVE_BUFFER_SIZE); if (rd == 0) { - continue; +...