| Message ID | 20220330181722.30280-1-jandryuk@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | libxl: constify libxl__stubdomain_is_linux | expand |
On Wed, Mar 30, 2022 at 02:17:22PM -0400, Jason Andryuk wrote: > libxl__stubdomain_is_linux can take a const pointer, so make the change. > > This isn't an issue in-tree, but was found with an OpenXT patch where it > was called with only const libxl_domain_build_info available. > > Signed-off-by: Jason Andryuk <jandryuk@gmail.com> Acked-by: Anthony PERARD <anthony.perard@citrix.com> Thanks,
diff --git a/tools/libs/light/libxl_internal.h b/tools/libs/light/libxl_internal.h index c0e7779d97..a26daec040 100644 --- a/tools/libs/light/libxl_internal.h +++ b/tools/libs/light/libxl_internal.h @@ -2326,7 +2326,7 @@ bool libxl__stubdomain_is_linux_running(libxl__gc *gc, uint32_t domid) } static inline -bool libxl__stubdomain_is_linux(libxl_domain_build_info *b_info) +bool libxl__stubdomain_is_linux(const libxl_domain_build_info *b_info) { /* right now qemu-tranditional implies MiniOS stubdomain and qemu-xen * implies Linux stubdomain */
libxl__stubdomain_is_linux can take a const pointer, so make the change. This isn't an issue in-tree, but was found with an OpenXT patch where it was called with only const libxl_domain_build_info available. Signed-off-by: Jason Andryuk <jandryuk@gmail.com> --- tools/libs/light/libxl_internal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)