From eebe543d68e43e1bd22fabd1cf1947b784176e50 Mon Sep 17 00:00:00 2001
From: Shamser Ahmed <shamser.ahmed@lexisnexis.com>
Date: Tue, 24 Sep 2024 16:39:41 +0100
Subject: [PATCH] Changes following review

Signed-off-by: Shamser Ahmed <shamser.ahmed@lexisnexis.com>
---
 dali/base/sysinfologger.cpp | 21 +++++++++++++--------
 1 file changed, 13 insertions(+), 8 deletions(-)

diff --git a/dali/base/sysinfologger.cpp b/dali/base/sysinfologger.cpp
index bbb1107a985..e0245fcf171 100644
--- a/dali/base/sysinfologger.cpp
+++ b/dali/base/sysinfologger.cpp
@@ -190,11 +190,7 @@ class CSysInfoLoggerMsgFilter : public CSimpleInterfaceOf<ISysInfoLoggerMsgFilte
     {
         if (hiddenOnly && visibleOnly)
             throw makeStringExceptionV(-1, "ISysInfoLoggerMsgFilter: cannot filter by both hiddenOnly and visibleOnly");
-        if (!_month && _day)
-            throw makeStringExceptionV(-1, "ISysInfoLoggerMsgFilter: month and year must be provided when filtering by day");
-        if (!_year && _month)
-            throw makeStringExceptionV(-1, "ISysInfoLoggerMsgFilter: year must be provided when filtering by month");
-       haveDateRange = false;
+        setDateRange(_year, _month, _day, _year, _month, _day);
     }
     virtual void setHiddenOnly() override
     {
@@ -238,6 +234,15 @@ class CSysInfoLoggerMsgFilter : public CSimpleInterfaceOf<ISysInfoLoggerMsgFilte
     }
     virtual void setDateRange(unsigned startYear, unsigned startMonth, unsigned startDay, unsigned endYear, unsigned endMonth, unsigned endDay) override
     {
+        if ( (startDay && (!startMonth||!startYear)) ||
+             (endDay && (!endMonth||!endYear)) )
+            throw makeStringExceptionV(-1, "ISysInfoLoggerMsgFilter: month and year must be provided when filtering by day");
+        if ((!startYear && startMonth) || (!endYear && endMonth))
+            throw makeStringExceptionV(-1, "ISysInfoLoggerMsgFilter: year must be provided when filtering by month");
+        // Make sure starts are on or before end dates
+        if ( (startYear > endYear) || (startMonth && (startYear == endYear && startMonth > endMonth))
+             || (startDay && (startYear == endYear && startMonth == endMonth && startDay > endDay)) )
+            throw makeStringExceptionV(-1, "ISysInfoLoggerMsgFilter: invalid date range: %04u-%02u-%02u to %04u-%02u-%02u", startYear, startMonth, startDay, endYear, endMonth, endDay);
         matchEndYear = endYear;
         matchEndMonth = endMonth;
         matchEndDay = endDay;
@@ -362,8 +367,8 @@ class CSysInfoLoggerMsgFilter : public CSimpleInterfaceOf<ISysInfoLoggerMsgFilte
     virtual StringBuffer & getQualifierXPathFilter(StringBuffer & xpath) const override
     {
         bool fullDayMatch=false;
-        bool hardMatchYear = matchStartYear==matchEndYear;
-        bool hardMatchMonth = matchStartMonth==matchEndMonth;
+        bool hardMatchYear = matchStartYear && (matchStartYear==matchEndYear);
+        bool hardMatchMonth = matchStartMonth && (matchStartMonth==matchEndMonth);
         if (hardMatchYear && hardMatchMonth)
         {
             xpath.appendf("m%04u%02u", matchStartYear, matchStartMonth);
@@ -433,7 +438,7 @@ class CSysInfoLoggerMsgIterator : public CSimpleInterfaceOf<ISysInfoLoggerMsgIte
 public:
     CSysInfoLoggerMsgIterator(IConstSysInfoLoggerMsgFilter * _filter, bool _updateable=false, IRemoteConnection *_conn=nullptr) : filter(_filter), updateable(_updateable), conn(_conn)
     {
-        if (conn==nullptr)
+        if (!conn)
         {
             unsigned mode = updateable ? RTM_LOCK_WRITE : RTM_LOCK_READ;
             conn.setown(querySDS().connect(SYS_INFO_ROOT, myProcessSession(), mode, SDS_LOCK_TIMEOUT));