We give the callers a const pointer to a static variable that we reuse between multiple calls: they're not supposed to free it, and they don't free it today. Fix the documentation and leave the still reachable pointer instead of reworking the function and its callers. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> --- lib/igt_fb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/igt_fb.c b/lib/igt_fb.c index 2f8968d..0462c35 100644 --- a/lib/igt_fb.c +++ b/lib/igt_fb.c @@ -1146,7 +1146,7 @@ const char *igt_format_str(uint32_t drm_format) * @format_count: pointer to integer to store the size of the allocated array * * This functions returns an array of all the drm fourcc codes supported by this - * library. The caller must free the allocated array again with free(). + * library. */ void igt_get_all_formats(const uint32_t **formats, int *format_count) { -- 2.7.0.rc3 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx