aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMattias Andrée <maandree@operamail.com>2014-05-25 13:14:23 +0200
committerMattias Andrée <maandree@operamail.com>2014-05-25 13:15:51 +0200
commit8c78c7950979ffae1bcdac75e11bd8f18cf320b6 (patch)
treec48302aa526f82625b0fce315d0e0db8b3e4c126 /src
parentm + add test that lists adjustment methds (diff)
downloadlibgamma-8c78c7950979ffae1bcdac75e11bd8f18cf320b6.tar.gz
libgamma-8c78c7950979ffae1bcdac75e11bd8f18cf320b6.tar.bz2
libgamma-8c78c7950979ffae1bcdac75e11bd8f18cf320b6.tar.xz
add buf_size parameter tolibgamma_list_methods
Signed-off-by: Mattias Andrée <maandree@operamail.com>
Diffstat (limited to 'src')
-rw-r--r--src/libgamma-facade.c32
-rw-r--r--src/libgamma-facade.h8
2 files changed, 24 insertions, 16 deletions
diff --git a/src/libgamma-facade.c b/src/libgamma-facade.c
index 25a7ba8..528a2fd 100644
--- a/src/libgamma-facade.c
+++ b/src/libgamma-facade.c
@@ -159,6 +159,9 @@ static int libgamma_list_method_test(int method, int operation)
* List available adjustment methods by their order of preference based on the environment
*
* @param methods Output array of methods, should be able to hold `GAMMA_METHOD_COUNT` elements
+ * @þaram buf_size The number of elements that fits in `methods`, it should be `GAMMA_METHOD_COUNT`,
+ * This is used to avoid writing outside the output buffer if this library adds new
+ * adjustment methods without the users of the library recompiling
* @param operation Allowed values:
* 0: Methods that the environment suggests will work, excluding fake.
* 1: Methods that the environment suggests will work, including fake.
@@ -166,9 +169,10 @@ static int libgamma_list_method_test(int method, int operation)
* 3: All real methods.
* 4: All methods.
* Other values invoke undefined behaviour.
- * @return The number of element that have been stored in `methods`
+ * @return The number of element that have been stored in `methods`, or should
+ * have been stored if the buffer was large enought
*/
-size_t libgamma_list_methods(int* restrict methods, int operation)
+size_t libgamma_list_methods(int* restrict methods, size_t buf_size, int operation)
{
#ifdef HAVE_NO_GAMMA_METHODS
(void) methods;
@@ -178,28 +182,28 @@ size_t libgamma_list_methods(int* restrict methods, int operation)
size_t n = 0;
#ifdef HAVE_GAMMA_METHOD_X_RANDR
- if (libgamma_list_method_test(GAMMA_METHOD_X_RANDR, operation))
- methods[n++] = GAMMA_METHOD_X_RANDR;
+ if (libgamma_list_method_test(GAMMA_METHOD_X_RANDR, operation) && (n++ < buf_size))
+ methods[n - 1] = GAMMA_METHOD_X_RANDR;
#endif
#ifdef HAVE_GAMMA_METHOD_X_VIDMODE
- if (libgamma_list_method_test(GAMMA_METHOD_X_VIDMODE, operation))
- methods[n++] = GAMMA_METHOD_X_VIDMODE;
+ if (libgamma_list_method_test(GAMMA_METHOD_X_VIDMODE, operation) && (n++ < buf_size))
+ methods[n - 1] = GAMMA_METHOD_X_VIDMODE;
#endif
#ifdef HAVE_GAMMA_METHOD_LINUX_DRM
- if (libgamma_list_method_test(GAMMA_METHOD_LINUX_DRM, operation))
- methods[n++] = GAMMA_METHOD_LINUX_DRM;
+ if (libgamma_list_method_test(GAMMA_METHOD_LINUX_DRM, operation) && (n++ < buf_size))
+ methods[n - 1] = GAMMA_METHOD_LINUX_DRM;
#endif
#ifdef HAVE_GAMMA_METHOD_W32_GDI
- if (libgamma_list_method_test(GAMMA_METHOD_W32_GDI, operation))
- methods[n++] = GAMMA_METHOD_W32_GDI;
+ if (libgamma_list_method_test(GAMMA_METHOD_W32_GDI, operation) && (n++ < buf_size))
+ methods[n - 1] = GAMMA_METHOD_W32_GDI;
#endif
#ifdef HAVE_GAMMA_METHOD_QUARTZ_CORE_GRAPHICS
- if (libgamma_list_method_test(GAMMA_METHOD_QUARTZ_CORE_GRAPHICS, operation))
- methods[n++] = GAMMA_METHOD_QUARTZ_CORE_GRAPHICS;
+ if (libgamma_list_method_test(GAMMA_METHOD_QUARTZ_CORE_GRAPHICS, operation) && (n++ < buf_size))
+ methods[n - 1] = GAMMA_METHOD_QUARTZ_CORE_GRAPHICS;
#endif
#ifdef HAVE_GAMMA_METHOD_DUMMY
- if (libgamma_list_method_test(GAMMA_METHOD_DUMMY, operation))
- methods[n++] = GAMMA_METHOD_DUMMY;
+ if (libgamma_list_method_test(GAMMA_METHOD_DUMMY, operation) && (n++ < buf_size))
+ methods[n - 1] = GAMMA_METHOD_DUMMY;
#endif
return n;
diff --git a/src/libgamma-facade.h b/src/libgamma-facade.h
index ecc3b13..f3d6ef8 100644
--- a/src/libgamma-facade.h
+++ b/src/libgamma-facade.h
@@ -77,6 +77,9 @@ typedef double libgamma_gamma_rampsd_fun(double encoding);
* List available adjustment methods by their order of preference based on the environment
*
* @param methods Output array of methods, should be able to hold `GAMMA_METHOD_COUNT` elements
+ * @þaram buf_size The number of elements that fits in `methods`, it should be `GAMMA_METHOD_COUNT`,
+ * This is used to avoid writing outside the output buffer if this library adds new
+ * adjustment methods without the users of the library recompiling
* @param operation Allowed values:
* 0: Methods that the environment suggests will work, excluding fake.
* 1: Methods that the environment suggests will work, including fake.
@@ -84,9 +87,10 @@ typedef double libgamma_gamma_rampsd_fun(double encoding);
* 3: All real methods.
* 4: All methods.
* Other values invoke undefined behaviour.
- * @return The number of element that have been stored in `methods`
+ * @return The number of element that have been stored in `methods`, or should
+ * have been stored if the buffer was large enought
*/
-size_t libgamma_list_methods(int* restrict methods, int operation);
+size_t libgamma_list_methods(int* restrict methods, size_t buf_size, int operation);
/**
* Return the capabilities of an adjustment method