fix: Spamming debug logs while resizing (#331)

The session id of Screen object might not be initialized when resized. Issue: linuxdeepin/developer-center#6296 Log: Spamming debug logs while resizing Signed-off-by: Yutao Meng <[email protected]>
linuxdeepin · Jan 25, 2024 · da0ce50 · da0ce50
1 parent a7ebec0
commit da0ce50
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/3rdparty/terminalwidget/lib/Screen.cpp b/3rdparty/terminalwidget/lib/Screen.cpp
@@ -98,6 +98,7 @@ Screen::Screen(int l, int c)
     , _lastPos(-1)
     , _lastDrawnChar(0)
     , _escapeSequenceUrlExtractor(new EscapeSequenceUrlExtractor)
+    , _sessionId(0)
 
 {
     _escapeSequenceUrlExtractor->setScreen(this);
@@ -380,7 +381,7 @@ void Screen::resizeImage(int new_lines, int new_columns)
         if (_currentTerminalDisplay) {
             // The 'zsh' works different from other shells when writting the command line.
             // It needs to identify the 'zsh' and calculate the new command line.
-            auto currSession = SessionManager::instance()->idToSession(_sessionId);
+            auto currSession = _sessionId > 0 ? SessionManager::instance()->idToSession(_sessionId) : nullptr;
             if (currSession
                     && (QLatin1String("zsh") == currSession->foregroundProcessName())
                     && (cursorLine > 0)