fix: 在插入新的timer时,边界条件判断不正确的问题 (DragonOS-Community#1091)

修复split_pos 初始值错误导致原本应该被插入到链表末尾的定时器,插入到了链表头 Signed-off-by: longjin <[email protected]>
Samuka007 · Mar 7, 2025 · 6fa06ff · 6fa06ff
1 parent 597315b
commit 6fa06ff
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -18,4 +18,5 @@ cppcheck.xml
 /target/
 Cargo.lock
 .cache
-compile_commands.json
+compile_commands.json
+/logs/
diff --git a/kernel/src/time/timer.rs b/kernel/src/time/timer.rs
@@ -157,7 +157,7 @@ impl Timer {
         let expire_jiffies = inner_guard.expire_jiffies;
         let self_arc = inner_guard.self_ref.upgrade().unwrap();
         drop(inner_guard);
-        let mut split_pos: usize = 0;
+        let mut split_pos: usize = timer_list.len();
         for (pos, elt) in timer_list.iter().enumerate() {
             if Arc::ptr_eq(&self_arc, &elt.1) {
                 warn!("Timer already in list");