logprof/genprof: glob generates wrong result in some cases

[Christian Boltz]

From a logprof run:

I used "glob with (e)xt" until I had
  4 - /home/**.jpg
 [5 - /**.jpg]

After that, I pressed "(g)lob", which resulted in
  4 - /home/**.jpg
  5 - /**.jpg
 [6 - /*]

Option 6 should be /**, not /*

[Kshitij Gupta (kgupta8592)]

Yes, this issue is mentioned as comments in the code in AppArmor.pm (line 3876) .

The programmer left a comment: # do we collapse to /* or /**

So apparently making this choice isn't a part of the code. Additional code has to be devised to identify this situation.

[Kshitij Gupta (kgupta8592)]

Sorry, that's line 3867.

[Christian Boltz (cboltz)]

The comment explains what the code does (which is a good thing because the regexes aren't easily readable).

Nevertheless, the code is slightly buggy. I'd _guess_ that we need to replace "/\*{1,2}" with "/\*{1,2}[^/]*" multiple times in lines 3866 to 3880.

[Christian Boltz (cboltz)]

Thy python utils have the correct behaviour - therefore fixed in >= 2.8.95 (aka 2.9 beta1).

I doubt someone will fix this minor issue in the old perl code, which makes it a wontfix for the 2.8 branch.