fix coredump by select + first

ilnurkh · ilnurkh · commit 40046403f568 · 2025-09-05T14:30:31.000+03:00
commit_hash:351845650cfb323aceb7b5ee0157adaba578705a
diff --git a/contrib/libs/jinja2cpp/patches/invalid-list-access-in-first.patch b/contrib/libs/jinja2cpp/patches/invalid-list-access-in-first.patch
@@ -0,0 +1,47 @@
+--- a/src/filters.cpp	(index)
++++ b/src/filters.cpp	(working tree)
+@@ -481,7 +481,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
+     switch (m_mode)
+     {
+         case FirstItemMode:
+-            if (listSize)
++            if (listSize && *listSize > 0)
+                 result = list.GetValueByIndex(0);
+             else
+             {
+@@ -491,7 +491,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
+             }
+             break;
+         case LastItemMode:
+-            if (listSize)
++            if (listSize && *listSize > 0)
+                 result = list.GetValueByIndex(listSize.value() - 1);
+             else
+             {
+@@ -502,7 +502,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
+             }
+             break;
+         case LengthMode:
+-            if (listSize)
++            if (listSize && *listSize > 0)
+                 result = static_cast<int64_t>(listSize.value());
+             else
+                 result = static_cast<int64_t>(std::distance(list.begin(), list.end()));
+@@ -511,7 +511,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
+         {
+             std::random_device rd;
+             std::mt19937 gen(rd());
+-            if (listSize)
++            if (listSize && *listSize > 0)
+             {
+                 std::uniform_int_distribution<> dis(0, static_cast<int>(listSize.value()) - 1);
+                 result = list.GetValueByIndex(dis(gen));
+@@ -548,7 +548,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
+         }
+         case ReverseMode:
+         {
+-            if (listSize)
++            if (listSize && *listSize > 0)
+             {
+                 auto size = listSize.value();
+                 InternalValueList resultList(size);
diff --git a/contrib/libs/jinja2cpp/src/filters.cpp b/contrib/libs/jinja2cpp/src/filters.cpp
@@ -481,7 +481,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
     switch (m_mode)
     {
         case FirstItemMode:
-            if (listSize)
+            if (listSize && *listSize > 0)
                 result = list.GetValueByIndex(0);
             else
             {
@@ -491,7 +491,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
             }
             break;
         case LastItemMode:
-            if (listSize)
+            if (listSize && *listSize > 0)
                 result = list.GetValueByIndex(listSize.value() - 1);
             else
             {
@@ -502,7 +502,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
             }
             break;
         case LengthMode:
-            if (listSize)
+            if (listSize && *listSize > 0)
                 result = static_cast<int64_t>(listSize.value());
             else
                 result = static_cast<int64_t>(std::distance(list.begin(), list.end()));
@@ -511,7 +511,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
         {
             std::random_device rd;
             std::mt19937 gen(rd());
-            if (listSize)
+            if (listSize && *listSize > 0)
             {
                 std::uniform_int_distribution<> dis(0, static_cast<int>(listSize.value()) - 1);
                 result = list.GetValueByIndex(dis(gen));
@@ -548,7 +548,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte
         }
         case ReverseMode:
         {
-            if (listSize)
+            if (listSize && *listSize > 0)
             {
                 auto size = listSize.value();
                 InternalValueList resultList(size);

Original file line number	Diff line number	Diff line change
`@@ -481,7 +481,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte`
`481`	`481`	`switch (m_mode)`
`482`	`482`	`{`
`483`	`483`	`case FirstItemMode:`
`484`		`- if (listSize)`
	`484`	`+ if (listSize && *listSize > 0)`
`485`	`485`	`result = list.GetValueByIndex(0);`
`486`	`486`	`else`
`487`	`487`	`{`
`@@ -491,7 +491,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte`
`491`	`491`	`}`
`492`	`492`	`break;`
`493`	`493`	`case LastItemMode:`
`494`		`- if (listSize)`
	`494`	`+ if (listSize && *listSize > 0)`
`495`	`495`	`result = list.GetValueByIndex(listSize.value() - 1);`
`496`	`496`	`else`
`497`	`497`	`{`
`@@ -502,7 +502,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte`
`502`	`502`	`}`
`503`	`503`	`break;`
`504`	`504`	`case LengthMode:`
`505`		`- if (listSize)`
	`505`	`+ if (listSize && *listSize > 0)`
`506`	`506`	`result = static_cast<int64_t>(listSize.value());`
`507`	`507`	`else`
`508`	`508`	`result = static_cast<int64_t>(std::distance(list.begin(), list.end()));`
`@@ -511,7 +511,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte`
`511`	`511`	`{`
`512`	`512`	`std::random_device rd;`
`513`	`513`	`std::mt19937 gen(rd());`
`514`		`- if (listSize)`
	`514`	`+ if (listSize && *listSize > 0)`
`515`	`515`	`{`
`516`	`516`	`std::uniform_int_distribution<> dis(0, static_cast<int>(listSize.value()) - 1);`
`517`	`517`	`result = list.GetValueByIndex(dis(gen));`
`@@ -548,7 +548,7 @@ InternalValue SequenceAccessor::Filter(const InternalValue& baseVal, RenderConte`
`548`	`548`	`}`
`549`	`549`	`case ReverseMode:`
`550`	`550`	`{`
`551`		`- if (listSize)`
	`551`	`+ if (listSize && *listSize > 0)`
`552`	`552`	`{`
`553`	`553`	`auto size = listSize.value();`
`554`	`554`	`InternalValueList resultList(size);`