search

bdraco / addon_securityadvisor

Old Security Advisor for cPanel 11.40 (upstream repo) -- Use CPANELINC/addon_securityadvisor
Other
16 stars 56 forks source link

Small fix for Kernel.pm #40

Closed demasu closed 11 years ago

demasu commented 11 years ago

Noticed an issue on my machine with the kernel checks. It claimed that I wasn't running the latest kernel, but that it was installed and recommended a reboot. It didn't go away after the reboot and I found it was due to ".x86_64" at the end of the string. I added 1 line that fixes that and doesn't seem to cause an issue. Please ignore the two test commits, I was having a small issue with git.

screen shot 2013-05-30 at 6 42 14 pm

bdraco commented 11 years ago

Would it be safer to just strip s/(?:x86_64|i.86)$//;

demasu commented 11 years ago

That does seem to be the safer option, yes. On May 30, 2013 7:03 PM, "J. Nick Koston" notifications@github.com wrote:

Would it be safer to just strip s/(?:x86_64|i.86)$//;

— Reply to this email directly or view it on GitHubhttps://github.com/bdraco/addon_securityadvisor/pull/40#issuecomment-18716257 .

bdraco commented 11 years ago

I stripped the arch from all the kernel versions in the latest version.

Thanks for pointing that out.