Re: [systemd-devel] [PATCH 2/2] coredump: Handle programs with spaces in COMM

2013-05-09 Thread Oleg Nesterov
Sorry for delay, vacation. On 05/03, Lennart Poettering wrote: > > On Wed, 01.05.13 18:42, Oleg Nesterov (o...@redhat.com) wrote: > > > I wouldn't say this is bug... at least this is expected. > > > > Sure, it is possible to rewrite format_corename/argv_split interaction, > > but this is a bit pai

Re: [systemd-devel] [PATCH 2/2] coredump: Handle programs with spaces in COMM

2013-05-04 Thread Colin Walters
On Fri, 2013-05-03 at 17:08 +0200, Lennart Poettering wrote: > It sounds really wrong to first merge this into one string and then > split it up again. It sounds much more sensible to instead just pass the > string array around all the time. What's the reason to make this one > string first? I'm

Re: [systemd-devel] [PATCH 2/2] coredump: Handle programs with spaces in COMM

2013-05-03 Thread Lennart Poettering
On Wed, 01.05.13 18:42, Oleg Nesterov (o...@redhat.com) wrote: > On 04/30, Colin Walters wrote: > > > > On Tue, 2013-04-30 at 19:47 +0200, Zbigniew Jędrzejewski-Szmek wrote: > > > On Tue, Apr 30, 2013 at 01:12:19PM -0400, Colin Walters wrote: > > > > This patch makes systemd-coredump handle proces

Re: [systemd-devel] [PATCH 2/2] coredump: Handle programs with spaces in COMM

2013-05-01 Thread Oleg Nesterov
On 04/30, Colin Walters wrote: > > On Tue, 2013-04-30 at 19:47 +0200, Zbigniew Jędrzejewski-Szmek wrote: > > On Tue, Apr 30, 2013 at 01:12:19PM -0400, Colin Walters wrote: > > > This patch makes systemd-coredump handle processes that have > > > whitespace in their COMM fields. > > > > > > fs/coredu