diff options
author | Tamito Kajiyama <rd6t-kjym@asahi-net.or.jp> | 2015-10-28 17:09:10 +0300 |
---|---|---|
committer | Tamito Kajiyama <rd6t-kjym@asahi-net.or.jp> | 2015-10-28 17:22:58 +0300 |
commit | efd774ce5a8536be526935ba118a10d7b2a843ef (patch) | |
tree | d87ea500bdcfa1ce448279a72ed97448755bad61 /source/blender/freestyle | |
parent | ced1c34f74d115f81052667d4d17e4efd6b282c4 (diff) |
Fix T44231: Freestyle causes crash on render.
The reported crash was confirmed as a segmentation fault in std::sort().
The cause of the crash was traced down to a binary comparison function
that was not satisfying the so-called strict weak ordering requirements of
the C++ standard sorting function. Specifically, the comparison operator
has to return false when two objects are equivalent (i.e., comp(a, a) must
be false), but that requirement was not met.
Since the binary comparison operator in question could be a user-defined
Python function, here a safety measure is implemented in the C++ layer to
make sure the aforementioned requirement is always satisfied.
Diffstat (limited to 'source/blender/freestyle')
-rw-r--r-- | source/blender/freestyle/intern/stroke/Operators.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/source/blender/freestyle/intern/stroke/Operators.cpp b/source/blender/freestyle/intern/stroke/Operators.cpp index 87ba34e8f42..dfb50d903f7 100644 --- a/source/blender/freestyle/intern/stroke/Operators.cpp +++ b/source/blender/freestyle/intern/stroke/Operators.cpp @@ -996,6 +996,8 @@ public: inline bool operator()(Interface1D *i1, Interface1D *i2) { + if (i1 == i2) + return false; if ((*_pred)(*i1, *i2) < 0) throw std::runtime_error("comparison failed"); return _pred->result; |