diff --git a/src/MaaCore/Task/Fight/MedicineCounterTaskPlugin.cpp b/src/MaaCore/Task/Fight/MedicineCounterTaskPlugin.cpp
index 7f7258ee15..4cfdd9612c 100644
--- a/src/MaaCore/Task/Fight/MedicineCounterTaskPlugin.cpp
+++ b/src/MaaCore/Task/Fight/MedicineCounterTaskPlugin.cpp
@@ -149,6 +149,13 @@ bool asst::MedicineCounterTaskPlugin::_run()
         if (m_expire_days > 0) {
             bool has_non_expiring = false;
             for (const auto& [use, _, expire_days, __] : using_medicine->medicines) {
+                if (use > 0 && expire_days == -1) {
+                    LogError << __FUNCTION__
+                             << "There are non-expiring medicines, and total count exceed max, need to reduce expire "
+                                "medicines first";
+                    has_non_expiring = true;
+                    break;
+                }
                 if (use > 0 && expire_days != -1 && expire_days > m_expire_days) {
                     has_non_expiring = true;
                     break;