intel / intel-cmt-cat

User space software for Intel(R) Resource Director Technology
http://www.intel.com/content/www/us/en/architecture-and-technology/resource-director-technology.html
Other
693 stars 183 forks source link

Fix Double quote array expansions to avoid re-splitting elements. #191

Closed wmealing closed 2 years ago

wmealing commented 3 years ago

Description

This is probably harmless as users will not be passing files to the program, but in the interest of future proofing this or possible unseen abuse, quoting it will solve a few of these issues for you.

This silences: https://github.com/koalaman/shellcheck/wiki/SC2068

Affected parts

Motivation and Context

I run shellchecks over the code on the system, its part of a mini audit that I am taking part of.

How Has This Been Tested?

This has had basic testing, I ran the pqos command, it worked as before... I tried abusing shell expansion it failed as it did before. Fedora 34 - x86-64 on A newer xeon CPU. This only really affects the shell side of the users interaction.

Types of changes

Checklist:

Thank you for your time. This is really minor and would understand if you dont want it.

Have a good day.