Sven Baars : mstask: Don't assume a ONCE trigger is on the current day.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 7f73494302b337eeb474879842ed8072ffcc8f2e
URL:    https://gitlab.winehq.org/wine/wine/-/commit/7f73494302b337eeb474879842ed807...

Author: Sven Baars 
Date:   Fri Dec  2 14:26:15 2022 +0100

mstask: Don't assume a ONCE trigger is on the current day.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52994

---

dlls/mstask/task.c               |  3 +++
 dlls/mstask/tests/task_trigger.c | 16 ++++++++++++++++
 2 files changed, 19 insertions(+)

diff --git a/dlls/mstask/task.c b/dlls/mstask/task.c
index d2a03aa8b56..3f3637f13a1 100644
--- a/dlls/mstask/task.c
+++ b/dlls/mstask/task.c
@@ -516,6 +516,9 @@ static HRESULT WINAPI MSTASK_ITask_GetNextRunTime(ITask *iface, SYSTEMTIME *rt)

case TASK_TIME_TRIGGER_ONCE:
                 st = current_st;
+                st.wYear = This->trigger[i].wBeginYear;
+                st.wMonth = This->trigger[i].wBeginMonth;
+                st.wDay = This->trigger[i].wBeginDay;
                 st.wHour = This->trigger[i].wStartHour;
                 st.wMinute = This->trigger[i].wStartMinute;
                 st.wSecond = 0;
diff --git a/dlls/mstask/tests/task_trigger.c b/dlls/mstask/tests/task_trigger.c
index 24b873770d2..236540cf0ee 100644
--- a/dlls/mstask/tests/task_trigger.c
+++ b/dlls/mstask/tests/task_trigger.c
@@ -525,6 +525,22 @@ static void test_GetNextRunTime(void)
     hr = ITaskTrigger_SetTrigger(trigger, &data);
     ok(hr == S_OK, "got %#lx\n", hr);

+    memset(&st, 0xff, sizeof(st));
+    hr = ITask_GetNextRunTime(task, &st);
+    ok(hr == S_OK, "got %#lx\n", hr);
+    ok(!memcmp(&st, &cmp, sizeof(st)), "got %u/%u/%u wday %u %u:%02u:%02u\n",
+       st.wDay, st.wMonth, st.wYear, st.wDayOfWeek,
+       st.wHour, st.wMinute, st.wSecond);
+
+    hr = ITaskTrigger_GetTrigger(trigger, &data);
+    ok(hr == S_OK, "got %#lx\n", hr);
+    data.rgFlags &= ~TASK_TRIGGER_FLAG_DISABLED;
+    data.TriggerType = TASK_TIME_TRIGGER_ONCE;
+    /* add 1 day to test triggers in the far future */
+    trigger_add_ms(&data, 24 * 60 * 60 * 1000, &cmp);
+    hr = ITaskTrigger_SetTrigger(trigger, &data);
+    ok(hr == S_OK, "got %#lx\n", hr);
+
     memset(&st, 0xff, sizeof(st));
     hr = ITask_GetNextRunTime(task, &st);
     ok(hr == S_OK, "got %#lx\n", hr);