Blob Blame History Raw
From 76b89df0efc0fe817d320b7b34b2b0530ffd5538 Mon Sep 17 00:00:00 2001
From: David King <dking@redhat.com>
Date: Fri, 4 Mar 2016 14:19:29 +0000
Subject: [PATCH] python: Avoid a crash on 64-bit systems

Annotate the return type of px_proxy_factory_new() to be void *, as otherwise
int is assumed. This works fine on 32-bit systems, where void * and int are the
same width, but is invalid on 64-bit (Linux) systems.

Additionally, annotate the argument type of free() and px_proxy_factory_free()
to be void * to match.

https://code.google.com/archive/p/libproxy/issues/146
---
 bindings/python/libproxy.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/bindings/python/libproxy.py b/bindings/python/libproxy.py
index cb75a4d..7bddba9 100644
--- a/bindings/python/libproxy.py
+++ b/bindings/python/libproxy.py
@@ -40,8 +40,12 @@ if platform.system() == "Windows":
 else:
     _libc = _load("c", 6)
 
+_libc.free.argtypes = ctypes.c_void_p,
+
 # Load libproxy
 _libproxy = _load("proxy", 1)
+_libproxy.px_proxy_factory_new.restype = ctypes.POINTER(ctypes.c_void_p)
+_libproxy.px_proxy_factory_free.argtypes = ctypes.c_void_p,
 _libproxy.px_proxy_factory_get_proxies.restype = ctypes.POINTER(ctypes.c_void_p)
 
 class ProxyFactory(object):
-- 
2.7.2