search for: guzik

...e phrase part. */ rfc2822_header_printf(msg, "To", "<%s>", reply_to); This should be work ok? Or You must change something? 2017-07-20 15:51 GMT+02:00 Stephan Bosch <stephan at rename-it.nl>: > > > Op 20-7-2017 om 10:37 schreef Kacper Guzik: > > Hi i have similiar problem like here: >> >> http://www.iredmail.org/forum/topic11833-iredmail-support-vo >> cation-respone-unknown-user.html >> >> >> email send : >> from : web123 at gmail.com >> to : jacka at mail.com >> >> voc...

...ake[2]: Leaving directory `/opt/pigeonhole-master/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/opt/pigeonhole-master' make: *** [all] Error 2 2017-08-24 2:13 GMT+02:00 Stephan Bosch <stephan at rename-it.nl>: > Op 7/20/2017 om 8:24 PM schreef Kacper Guzik: > > I have this version with FIXME > > > > > > /* FIXME: If From header of message has same address, we should > use > > that > > > > * instead to properly include the phrase part. > > > > */ > > > >...

On Tue, Dec 01, 2015 at 04:16:57PM +0100, Pino Toscano wrote: > On Tuesday 01 December 2015 15:59:56 Mateusz Guzik wrote: > > I would argue that /dev has to be at least partially populated for anything > > that gets executed in the chroot. At the very least special nodes like null, > > zero and {u,}random are needed. > > We do not assume anything about guests, which could be Linux with...

Amit Shah <amit.shah at redhat.com> writes: > Mateusz Guzik points out that we update the 'file' struct's private_data > field before we've successfully done all our checks. This means we can > return an error with the private_data field updated. This could lead to > problems. > > Fix by moving the assignment after all check...

Amit Shah <amit.shah at redhat.com> writes: > Mateusz Guzik points out that we update the 'file' struct's private_data > field before we've successfully done all our checks. This means we can > return an error with the private_data field updated. This could lead to > problems. > > Fix by moving the assignment after all check...

...mit.shah at redhat.com> writes: > There's a window between find_port_by_devt() returning a port and us > taking a kref on the port, where the port could get unplugged. Fix it > by taking the reference in find_port_by_devt() itself. > > Problem reported and analyzed by Mateusz Guzik. This fix is clearly correct, but what about the other find_port_by_* functions? Applied, Rusty. > > CC: <stable at vger.kernel.org> > Reported-by: Mateusz Guzik <mguzik at redhat.com> > Signed-off-by: Amit Shah <amit.shah at redhat.com> > --- > drivers/char/...

...mit.shah at redhat.com> writes: > There's a window between find_port_by_devt() returning a port and us > taking a kref on the port, where the port could get unplugged. Fix it > by taking the reference in find_port_by_devt() itself. > > Problem reported and analyzed by Mateusz Guzik. This fix is clearly correct, but what about the other find_port_by_* functions? Applied, Rusty. > > CC: <stable at vger.kernel.org> > Reported-by: Mateusz Guzik <mguzik at redhat.com> > Signed-off-by: Amit Shah <amit.shah at redhat.com> > --- > drivers/char/...

On Tue, Dec 01, 2015 at 03:59:56PM +0100, Mateusz Guzik wrote: > CHROOT_IN/OUT around commandvf are definitely problematic. chroot should be > done in the child, which also removes the need to chroot out in the > parent. The CHROOT_IN/OUT business does need to be rewritten. Every instance where we currently do something like: CHROOT_IN;...

Op 8/24/2017 om 7:05 PM schreef Kacper Guzik: > i Cant compile piegeon hole from github with this feature: > That's because this branch is based on master (v2.3). I created a rebased branch for v2.2: https://github.com/dovecot/pigeonhole/tree/vacation-to-header-ignore-envelope-2.2 Regards, Stephan. > 2822.c -fPIC -DPIC -...

...executing the process if they fail - open /dev/null without O_CLOEXEC, so it stays available for the exec'ed process, and thus we don't need to provide an own fd for stdin Followup of commit fd2f175ee79d29df101d353e2f380db27b19553a, thanks also to the notes and hints provided by Mateusz Guzik. --- daemon/command.c | 17 ++--------------- daemon/daemon.h | 1 + daemon/guestfsd.c | 39 +++++++++++++++++++++++++++++++-------- 3 files changed, 34 insertions(+), 23 deletions(-) diff --git a/daemon/command.c b/daemon/command.c index 27a4d0c..c4efa5b 100644 --- a/daemon/command.c +++ b/d...

...error unless extraterrestial circumstances occur, and even then the fd is no longer in use by the process. As such, I would argue checking for errors here is not necessary. Note that prior sample does not check. In case an error was returned, the code fails to waitpid() for the child. -- Mateusz Guzik

...best points obtained by the 'leaps' package. The specific questions are: x. How can I Convert a data frame's column into a vector? x. Now that I have the vectors, how can I specify a linear model of a linear regression with associated variances? Thanks a lot in advance!! Alan Aspuru-Guzik -.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.- r-help mailing list -- Read http://www.ci.tuwien.ac.at/~hornik/R/R-FAQ.html Send "info", "help", or "[un]subscribe" (in the "body", not the subject !) To: r-help-request at stat...

On Tue, Dec 01, 2015 at 04:58:11PM +0000, Richard W.M. Jones wrote: > On Tue, Dec 01, 2015 at 03:59:56PM +0100, Mateusz Guzik wrote: > > CHROOT_IN/OUT around commandvf are definitely problematic. chroot should be > > done in the child, which also removes the need to chroot out in the > > parent. > > The CHROOT_IN/OUT business does need to be rewritten. Every > instance where we currently do so...

Op 7/20/2017 om 8:24 PM schreef Kacper Guzik: > I have this version with FIXME > > > /* FIXME: If From header of message has same address, we should use > that > > * instead to properly include the phrase part. > > */ > > rfc2822_header_printf(msg, "To", "<%s...

Mateusz Guzik points out that we update the 'file' struct's private_data field before we've successfully done all our checks. This means we can return an error with the private_data field updated. This could lead to problems. Fix by moving the assignment after all checks are done. CC: <sta...

There's a window between find_port_by_devt() returning a port and us taking a kref on the port, where the port could get unplugged. Fix it by taking the reference in find_port_by_devt() itself. Problem reported and analyzed by Mateusz Guzik. CC: <stable at vger.kernel.org> Reported-by: Mateusz Guzik <mguzik at redhat.com> Signed-off-by: Amit Shah <amit.shah at redhat.com> --- drivers/char/virtio_console.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/char/virtio_console.c...

Hi i have similiar problem like here: http://www.iredmail.org/forum/topic11833-iredmail-support-vocation-respone-unknown-user.html email send : from : web123 at gmail.com to : jacka at mail.com vocation sent back from : jacka at mail.com to : srs0=hmc8=v7=gmail.com=web123 at mail.com postsrsd changing return-path from web123 at gmail.com to srs0=hmc8=v7= gmail.com=web123 at mail.com this

...o indicate a port is already open by another process. Use EBUSY for that. This does change what we report to userspace, but I believe userspace can look at it this way: it gets EBUSY, a new error code, instead of EMFILE. It's still an error, and that's not changing. Reported-by: Mateusz Guzik <mguzik at redhat.com> Signed-off-by: Amit Shah <amit.shah at redhat.com> --- Rusty, is this OK? It's a change for the userspace, so is it considered breakage? OTOH the current return value is obviously wrong, 'Broken' userspace code could be relying on the exact error t...

...o indicate a port is already open by another process. Use EBUSY for that. This does change what we report to userspace, but I believe userspace can look at it this way: it gets EBUSY, a new error code, instead of EMFILE. It's still an error, and that's not changing. Reported-by: Mateusz Guzik <mguzik at redhat.com> Signed-off-by: Amit Shah <amit.shah at redhat.com> --- Rusty, is this OK? It's a change for the userspace, so is it considered breakage? OTOH the current return value is obviously wrong, 'Broken' userspace code could be relying on the exact error t...

...they fail > - open /dev/null without O_CLOEXEC, so it stays available for the > exec'ed process, and thus we don't need to provide an own fd for stdin > > Followup of commit fd2f175ee79d29df101d353e2f380db27b19553a, thanks also > to the notes and hints provided by Mateusz Guzik. Looks good, thanks. I only have optional nits. > --- > daemon/command.c | 17 ++--------------- > daemon/daemon.h | 1 + > daemon/guestfsd.c | 39 +++++++++++++++++++++++++++++++-------- > 3 files changed, 34 insertions(+), 23 deletions(-) > > diff --git a/daemon/comma...