On Tue, Jul 14, 2020 at 06:42:56PM +0200, Philippe Mathieu-Daudé wrote: > Document qemu_find_file(), in particular the returned > value which must be freed. > > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> > --- > include/qemu-common.h | 14 ++++++++++++++ > 1 file changed, 14 insertions(+)
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> > > diff --git a/include/qemu-common.h b/include/qemu-common.h > index d0142f29ac..d6a08259d3 100644 > --- a/include/qemu-common.h > +++ b/include/qemu-common.h > @@ -110,6 +110,20 @@ const char *qemu_get_vm_name(void); > > #define QEMU_FILE_TYPE_BIOS 0 > #define QEMU_FILE_TYPE_KEYMAP 1 > +/** > + * qemu_find_file: > + * @type: QEMU_FILE_TYPE_BIOS (for BIOS, VGA BIOS) > + * or QEMU_FILE_TYPE_KEYMAP (for keymaps). > + * @name: File name s/File name/relative or absolute file name/ > + * Insert: "If @name exists on disk as an absolute path, or a path relative to the current directory, then returns @name unchanged. > + * Search for @name file in the data directories, either configured at > + * build time (DATADIR) or registered with the -L command line option. s/Search/Otherwise searches/ > + * > + * The caller must use g_free() to free the returned data when it is > + * no longer required. > + * > + * Returns: absolute path to the file or NULL on error. It doesn't always return an absolute path. It can return a path relative to the currently working directory via the initial short-circuit access() check. Returns: a path that can access @name, or NULL if no matching file exists. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|