admin/idapython-src
1
0
Fork 0
mirror of https://github.com/idapython/src synced 2026-06-08 14:47:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

+ bugfix: IDAPython: Choose.OnGetLineAttr was broken

Browse Source 
- It was only accepting lists (it should accept both tuples and lists)
This commit is contained in:
Elias Bachaalany
2023-01-12 21:27:42 -08:00
parent ca0fb1a4c2
commit 02c95291c3
1 changed files with 11 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pywraps/py_kernwin_choose.hpp
+11 -8
View File
@@ -478,15 +478,18 @@ void py_chooser_mixin_t::mixin_get_row(
PyObject_CallMethod(
self.o, (char *)S_ON_GET_LINE_ATTR,
"i", int(n)));
if ( PyErr_Occurred() != nullptr )
return;
if ( pyres.result != nullptr && PyList_Check(pyres.result.o) )
if ( PyErr_Occurred() == nullptr && pyres.result != nullptr && PySequence_Check(pyres.result.o) )
{
PyObject *item;
if ( (item = PyList_GetItem(pyres.result.o, 0)) != nullptr )
attrs->color = PyInt_AsLong(item);
if ( (item = PyList_GetItem(pyres.result.o, 1)) != nullptr )
attrs->flags = PyInt_AsLong(item);
{
newref_t item(PySequence_GetItem(pyres.result.o, 0));
if (item.o != nullptr && PyLong_Check(item.o))
attrs->color = PyLong_AsUnsignedLong(item.o);
}
{
newref_t item(PySequence_GetItem(pyres.result.o, 1));
if (item.o != nullptr && PyLong_Check(item.o))
attrs->flags = PyInt_AsLong(item.o);
}
}
}
}