Kernel path detection in image files

[Javier Collado]

This is a follow-up of:
https://code.launchpad.net/~nuclearbob/utah/iso-static-analysis-updates/+merge/138164

I'm concerned about the words `kernel` or `linux` appear anywhere in a line and getting a string that isn't a real kernel file path.

The proposed alternative is that instead of this:

if 'kernel' in line or 'linux' in line:
    newkernel = line.split()[1].strip('./')

We use something like this:

import re

...

match = re.match('^\s+(kernel|linux)\s+(?P<path>[^\s]+).*', line)
if match:
    newkernel = match.group('path')

[Dimitri John Ledkov (xnox)]

Ah, I see the rational of mis-detection.
So:
if line.split()[0] in ('kernel', 'linux'):
    newkernel = line.split()[1].strip('./')

Should satisfy that requirement? Split strips leading whitespace.
We could even introduce a temp variable to hold line.spit() list.....

[Javier Collado (javier.collado)]

Yes, I'd be happy with that (including the assignment of a temporary variable).
I think Max will like also your approach, so I'll create a merge proposal with
it.

Thanks for your feedback.