diff options
| author | Rui Matos <tiagomatos@gmail.com> | 2015-05-21 17:53:17 +0200 |
|---|---|---|
| committer | Rui Matos <tiagomatos@gmail.com> | 2015-07-17 15:17:36 +0200 |
| commit | deb3abcd8a11328979ac31c4e48a3754c4b4b37f (patch) | |
| tree | 5e4c32af481c36c9c5dd80d91887a71e6af8aff0 | |
| parent | 5cfe5b500870cd65386bff91ec12e7f263831f97 (diff) | |
| download | pygobject-3-16.tar.gz | |
Avoid a silent long to int truncationpygobject-3-16
If the python object contains a value bigger than MAXUINT we'd
silently truncate it when assigning to 'val' and the if condition
would always be true.
This was caught by a coverity scan.
https://bugzilla.gnome.org/show_bug.cgi?id=749698
| -rw-r--r-- | gi/pygi-value.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gi/pygi-value.c b/gi/pygi-value.c index 9d5d0caf..7fdf7675 100644 --- a/gi/pygi-value.c +++ b/gi/pygi-value.c @@ -382,7 +382,7 @@ pyg_value_from_pyobject_with_error(GValue *value, PyObject *obj) case G_TYPE_UINT: { if (PYGLIB_PyLong_Check(obj)) { - guint val; + gulong val; /* check that number is not negative */ if (PyLong_AsLongLong(obj) < 0) @@ -390,7 +390,7 @@ pyg_value_from_pyobject_with_error(GValue *value, PyObject *obj) val = PyLong_AsUnsignedLong(obj); if (val <= G_MAXUINT) - g_value_set_uint(value, val); + g_value_set_uint(value, (guint) val); else return -1; } else { |