Mini-OS: fix 9pfront fix

[PATCH] Mini-OS: fix 9pfront fix

Posted by Juergen Gross 7 months, 1 week ago

There was a typo in the recent 9pfront fix.

Fixes: b119f0178fd8 ("Mini-OS: fix 9pfs frontend error path")
Reported-by: Julien Grall <julien@xen.org>
Signed-off-by: Juergen Gross <jgross@suse.com>
---
 9pfront.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/9pfront.c b/9pfront.c
index 042879a7..1741d600 100644
--- a/9pfront.c
+++ b/9pfront.c
@@ -729,7 +729,7 @@ static int p9_stat(struct dev_9pfs *dev, uint32_t fid, struct p9_stat *stat)
 
     ret = req->result;
     if ( ret )
-        free_stat(&stat);
+        free_stat(stat);
 
     put_free_req(dev, req);
 
-- 
2.35.3

Re: [PATCH] Mini-OS: fix 9pfront fix

Posted by Samuel Thibault 7 months, 1 week ago

Juergen Gross, le mer. 07 févr. 2024 14:49:20 +0100, a ecrit:
> There was a typo in the recent 9pfront fix.
> 
> Fixes: b119f0178fd8 ("Mini-OS: fix 9pfs frontend error path")
> Reported-by: Julien Grall <julien@xen.org>
> Signed-off-by: Juergen Gross <jgross@suse.com>

Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>

> ---
>  9pfront.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/9pfront.c b/9pfront.c
> index 042879a7..1741d600 100644
> --- a/9pfront.c
> +++ b/9pfront.c
> @@ -729,7 +729,7 @@ static int p9_stat(struct dev_9pfs *dev, uint32_t fid, struct p9_stat *stat)
>  
>      ret = req->result;
>      if ( ret )
> -        free_stat(&stat);
> +        free_stat(stat);
>  
>      put_free_req(dev, req);
>  
> -- 
> 2.35.3
>