commit 58e3c8bc7246f78df3d257cf376c37d004d559e1
author Семён Марьясин <simeon@maryasin.name> Sat Feb 16 20:29:53 2019 +0300
committer Nguyen Anh Quynh <aquynh@gmail.com> Sun Feb 17 01:29:53 2019 +0800
tree 06cda4193d529902b4c07f005d848cd915240708
parent 6782c481662d2bbcd2a08a0f693845299584b8a8

Fix skipdata struct being destroyed (#1385)

bindings/python/capstone/__init__.py

diff --git a/bindings/python/capstone/__init__.py b/bindings/python/capstone/__init__.py
index 203a807..d6fe464 100644
--- a/bindings/python/capstone/__init__.py
+++ b/bindings/python/capstone/__init__.py
@@ -834,6 +834,9 @@
         # default mnemonic for SKIPDATA
         self._skipdata_mnem = ".byte"
         self._skipdata_cb = (None, None)
+        # store reference to option object to avoid it being freed
+        # because C code uses it by reference
+        self._skipdata_opt = _cs_opt_skipdata()
         self._skipdata = False
 
 
@@ -908,12 +911,11 @@
 
     @skipdata_setup.setter
     def skipdata_setup(self, opt):
-        _skipdata_opt = _cs_opt_skipdata()
         _mnem, _cb, _ud = opt
-        _skipdata_opt.mnemonic = _mnem.encode()
-        _skipdata_opt.callback = CS_SKIPDATA_CALLBACK(_cb or 0)
-        _skipdata_opt.user_data = ctypes.cast(_ud, ctypes.c_void_p)
-        status = _cs.cs_option(self.csh, CS_OPT_SKIPDATA_SETUP, ctypes.cast(ctypes.byref(_skipdata_opt), ctypes.c_void_p))
+        self._skipdata_opt.mnemonic = _mnem.encode()
+        self._skipdata_opt.callback = CS_SKIPDATA_CALLBACK(_cb or 0)
+        self._skipdata_opt.user_data = ctypes.cast(_ud, ctypes.c_void_p)
+        status = _cs.cs_option(self.csh, CS_OPT_SKIPDATA_SETUP, ctypes.cast(ctypes.byref(self._skipdata_opt), ctypes.c_void_p))
         if status != CS_ERR_OK:
             raise CsError(status)