[AT-SPI] Fixed uint underflow risk 59/245359/10
authorMaria Bialota <m.bialota@samsung.com>
Wed, 7 Oct 2020 14:33:40 +0000 (16:33 +0200)
committerArtur Świgoń <a.swigon@samsung.com>
Fri, 23 Oct 2020 13:07:01 +0000 (15:07 +0200)
Change-Id: I7967a7ccf9cfe73beb147f41d2c1850f7ee63da5

dali-toolkit/internal/controls/text-controls/text-editor-impl.cpp
dali-toolkit/internal/controls/text-controls/text-field-impl.cpp
dali-toolkit/internal/controls/text-controls/text-label-impl.cpp

index 7b4e8a2f5d21321fa110289c0630cd53487869e6..4650a6dccc393a1931038ff7d112191f5f7d2be9 100644 (file)
@@ -1999,7 +1999,7 @@ Dali::Accessibility::Range TextEditor::AccessibleImpl::GetTextAtOffset(
             if (boundary == Dali::Accessibility::TextBoundary::LINE)
               counter++;
           }
-          if ((counter - 1) == offset)
+          if ((counter > 0) && ((counter - 1) == offset))
           {
             range.content = txt.substr(start, index - start + 1);
             range.startOffset = start;
index 1d79c5feb68b471cd231bb393feb54557d87ad7e..312a54e8bcdb21e89cfe9227d454ff1a706ec556 100644 (file)
@@ -1936,7 +1936,7 @@ Dali::Accessibility::Range TextField::AccessibleImpl::GetTextAtOffset(
             if (boundary == Dali::Accessibility::TextBoundary::LINE)
               counter++;
           }
-          if ((counter - 1) == offset)
+          if ((counter > 0) && ((counter - 1) == offset))
           {
             range.content = txt.substr(start, index - start + 1);
             range.startOffset = start;
index d02fd675094600d0dc10632c9690f46e128899c4..3ecb376a76eea3fc4354e212891b4384a5c355d0 100755 (executable)
@@ -1077,7 +1077,7 @@ Dali::Accessibility::Range TextLabel::AccessibleImpl::GetTextAtOffset(
             if (boundary == Dali::Accessibility::TextBoundary::LINE)
               counter++;
           }
-          if ((counter - 1) == offset)
+          if ((counter > 0) && ((counter - 1) == offset))
           {
             range.content = txt.substr(start, index - start + 1);
             range.startOffset = start;