diff mbox series

fwts_efi_module: constify read-only argument in check_module_loaded_no_dev

Message ID 20210411132859.1189798-1-colin.king@canonical.com
State Accepted
Headers show
Series fwts_efi_module: constify read-only argument in check_module_loaded_no_dev | expand

Commit Message

Colin Ian King April 11, 2021, 1:28 p.m. UTC
From: Colin Ian King <colin.king@canonical.com>

Make a read-only function pointer argument const.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 src/lib/src/fwts_efi_module.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alex Hung April 11, 2021, 8:54 p.m. UTC | #1
On 2021-04-11 7:28 a.m., Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
> 
> Make a read-only function pointer argument const.
> 
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/lib/src/fwts_efi_module.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/lib/src/fwts_efi_module.c b/src/lib/src/fwts_efi_module.c
> index 5669f863..3756d614 100644
> --- a/src/lib/src/fwts_efi_module.c
> +++ b/src/lib/src/fwts_efi_module.c
> @@ -36,7 +36,7 @@ static char *module_name = NULL;
>   */
>  static int check_module_loaded_no_dev(
>  	fwts_framework *fw,
> -	char *module)
> +	const char *module)
>  {
>  	bool loaded;
>  
> 


Acked-by: Alex Hung <alex.hung@canonical.com>
Ivan Hu April 12, 2021, 7:33 a.m. UTC | #2
On 4/11/21 9:28 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
> 
> Make a read-only function pointer argument const.
> 
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/lib/src/fwts_efi_module.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/lib/src/fwts_efi_module.c b/src/lib/src/fwts_efi_module.c
> index 5669f863..3756d614 100644
> --- a/src/lib/src/fwts_efi_module.c
> +++ b/src/lib/src/fwts_efi_module.c
> @@ -36,7 +36,7 @@ static char *module_name = NULL;
>   */
>  static int check_module_loaded_no_dev(
>  	fwts_framework *fw,
> -	char *module)
> +	const char *module)
>  {
>  	bool loaded;
>  
> 

Acked-by: Ivan Hu <ivan.hu@canonical.com>
diff mbox series

Patch

diff --git a/src/lib/src/fwts_efi_module.c b/src/lib/src/fwts_efi_module.c
index 5669f863..3756d614 100644
--- a/src/lib/src/fwts_efi_module.c
+++ b/src/lib/src/fwts_efi_module.c
@@ -36,7 +36,7 @@  static char *module_name = NULL;
  */
 static int check_module_loaded_no_dev(
 	fwts_framework *fw,
-	char *module)
+	const char *module)
 {
 	bool loaded;