lkml.org 
[lkml]   [2014]   [Mar]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2] initramfs: print error and shell out for unsupported content
Am 22.03.2014 00:07, schrieb Alexander Holler:
> Am 21.03.2014 23:55, schrieb Andrew Morton:

>> Should we check for \t and \n as well?
>
> Hmm, maybe. But usually there aren't filenames wich do contain those
> characters, and if you want to break (or exploit) the kernel build
> process, there are easier ways. But colons and spaces are more widely
> used, e.g. the colons in my initramfs were generated by bluez (look at
> /var/lib/bluetooth).
>
> I think the current process is good enough for most stuff one wants to
> put into an initramfs, and it has the great feature of the uid/guid
> translation.
> So just a quick check to avoid the most basic problems should be ok. And
> I don't really see a need to check for \t and \n too, because nobody
> sane uses them in filenames. But ok, that just would be a few chars more
> in the regex for find. ;)
>
> I leave that up to you.

Sorry for that answer, but I dislike the process which turns patch
posters into remote keyboards quiet a lot.

When I post a patch I usually already have spend time to discover,
investigate, describe, fix the problem (as good as I can or I'm
willing), write a checkpatch conforming patch, and test that patch,
which most of the time sums up to several hours (I didn't want to spend
at all).

And then all kind of comments are arriving to change this and that and
fix that typo or remove that space and the patch ping pong starts until
an arbitrary maintainer or reviewer is happy.

I think the process should be more like that:

- patch posted
(- comments from maintainer)
- original patch commited
- patch from maintainer or reviewer on top of the original patch commited

That would give credits to both, the original patch poster and the
maintainer or reviewer (for his changes).

Of course, that isn't an ideal solution, but I would like it a lot, if
such a process would at least be seen as a possible and reasonable way,
even if it means that two patches instead of one do end up in the
repository.
To keep things together, maybe the second patch could be marked as
[maintainer patch] or [reviewer patch], that would make it clear that
both patches tie together and should be reverted both, if needed.

Anyway, I'm drifting offtopic, but that was something I always wanted to
suggest.

But the real reason why I've started to write this mail, is that it
might make sense to delete usr/.initramfs_data.cpio.d with a make
(dist)clean. That file is the reason why the bug with colons might drive
people crazy, because not even a make distclean (or git clean -df) will
make make working again after a broken usr/.initramfs_data.cpio.d was
build (with colons in filenames).

Maybe I will post a patch for that too, if I'm willing to do and test it
(together with a v3 of the patch which checks for \t \n and \r too).

Regards,

Alexander Holler



\
 
 \ /
  Last update: 2014-03-22 11:21    [W:0.104 / U:0.724 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site