search

ryran / xsos

sosreport examiner - instantly summarize system info from a sosreport or a running system
337 stars 121 forks source link

interrupts: fix a regex #244

Closed chaerin16 closed 5 years ago

chaerin16 commented 5 years ago

I found a bug in the part of interrupts.

In case there is a line like below in /proc/interrupts, 29: 0 0 0 0 0 0 0 0 IR-PCI-MSI-edge 0000:00:11.5

I expected the output will be as below. 29: ........ IR-PCI-MSI-edge 0000:00:11.5

But, actual output is as below. 29: ........ IR-PCI-MSI-edge▊

I checked the source code, and I found the regex is wrong. According to the source code, it outputs the field as it is only in case the field contains alphabet. But actually '.' or ':' can be located there, too. So, I think the regex would be better to change [a-zA-Z] to [^0-9] like this PR.

Thank you.

superjamie commented 5 years ago

Thanks for this, it works well.

I modified your commit to increment the version number and applied with 5bd299320