diff options
| author | Alin Andrei Abahnencei <alinandrei2004@gmail.com> | 2024-12-23 10:57:38 +0200 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2025-01-09 13:15:32 +0100 |
| commit | e08ed1d92f083cc604cb25eb284deb21016268fd (patch) | |
| tree | 834f67de08df8444f50372c114e80abdbed2e720 /basic | |
| parent | 1e06592574472cea476cf2d505d53c7838004fc0 (diff) | |
tdf#154285 Check upper bound of arguments in SbRtl_Year function
Signed-off-by: Alin Andrei Abahnencei <alinandrei2004@gmail.com>
Change-Id: Iffffe3988afdc91068e85beaafc5ff1580210ec6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/179253
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'basic')
| -rw-r--r-- | basic/source/runtime/methods.cxx | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/basic/source/runtime/methods.cxx b/basic/source/runtime/methods.cxx index 0982c591de6c..17b9029b9d72 100644 --- a/basic/source/runtime/methods.cxx +++ b/basic/source/runtime/methods.cxx @@ -2031,15 +2031,10 @@ void SbRtl_Day(StarBASIC *, SbxArray & rPar, bool) void SbRtl_Year(StarBASIC *, SbxArray & rPar, bool) { - if (rPar.Count() < 2) - { - StarBASIC::Error( ERRCODE_BASIC_BAD_ARGUMENT ); - } - else - { - sal_Int16 nYear = implGetDateYear(rPar.Get(1)->GetDate()); - rPar.Get(0)->PutInteger(nYear); - } + if (rPar.Count() != 2) + return StarBASIC::Error( ERRCODE_BASIC_BAD_ARGUMENT ); + sal_Int16 nYear = implGetDateYear(rPar.Get(1)->GetDate()); + rPar.Get(0)->PutInteger(nYear); } sal_Int16 implGetHour( double dDate ) |