[PATCH v3] bus/fslmc/dpaa2: replace system("echo ...") with file i/o
Thomas Monjalon
thomas at monjalon.net
Thu Oct 17 18:15:39 CEST 2024
26/08/2024 18:11, Stephen Hemminger:
> Using system() is a bad idea in driver code because it introduces
> a number of potential security issues. The codeql analysis tool
> flags this a potential security issue.
>
> Instead just use normal stdio to do the same thing.
>
> Compile test only, do not have this hardware and therefore can
> not test this.
>
> Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
> Reviewed-by: Sachin Saxena <sachin.saxena at oss.nxp.com>
Applied, thanks.
More information about the dev
mailing list