gem5-dev@gem5.org

The gem5 Developer List

View all threads

[XS] Change in gem5/gem5[develop]: base: Fix VNC initilization

YL
Yen-lin Lai (Gerrit)
Thu, Apr 13, 2023 2:50 AM

Yen-lin Lai has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/69757?usp=email )

Change subject: base: Fix VNC initilization
......................................................................

base: Fix VNC initilization

In a previous commit, the initialization of dataFd to -1 is removed.
Add it back so VNC server can properly accept connection.

Fixes: 52775185b2 ("base,cpu,dev,sim: Pull common logic into
ListenSocket::listen().")
Change-Id: I4246d1fddc766cb190a04d4f984fc1ce73af3fb0

M src/base/vnc/vncserver.cc
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/base/vnc/vncserver.cc b/src/base/vnc/vncserver.cc
index f342419..4b1ddae 100644
--- a/src/base/vnc/vncserver.cc
+++ b/src/base/vnc/vncserver.cc
@@ -117,7 +117,7 @@
*/
VncServer::VncServer(const Params &p)
: VncInput(p), listenEvent(NULL), dataEvent(NULL), number(p.number),

  •  listener(p.port.build(p.name)),
    
  •  dataFd(-1), listener(p.port.build(p.name)),
      sendUpdate(false), supportsRawEnc(false), supportsResizeEnc(false)
    
    {
    if (p.port)

--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/69757?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I4246d1fddc766cb190a04d4f984fc1ce73af3fb0
Gerrit-Change-Number: 69757
Gerrit-PatchSet: 1
Gerrit-Owner: Yen-lin Lai yenlinlai@google.com
Gerrit-MessageType: newchange

Yen-lin Lai has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/69757?usp=email ) Change subject: base: Fix VNC initilization ...................................................................... base: Fix VNC initilization In a previous commit, the initialization of dataFd to -1 is removed. Add it back so VNC server can properly accept connection. Fixes: 52775185b2 ("base,cpu,dev,sim: Pull common logic into ListenSocket::listen().") Change-Id: I4246d1fddc766cb190a04d4f984fc1ce73af3fb0 --- M src/base/vnc/vncserver.cc 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/base/vnc/vncserver.cc b/src/base/vnc/vncserver.cc index f342419..4b1ddae 100644 --- a/src/base/vnc/vncserver.cc +++ b/src/base/vnc/vncserver.cc @@ -117,7 +117,7 @@ */ VncServer::VncServer(const Params &p) : VncInput(p), listenEvent(NULL), dataEvent(NULL), number(p.number), - listener(p.port.build(p.name)), + dataFd(-1), listener(p.port.build(p.name)), sendUpdate(false), supportsRawEnc(false), supportsResizeEnc(false) { if (p.port) -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/69757?usp=email To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings Gerrit-Project: public/gem5 Gerrit-Branch: develop Gerrit-Change-Id: I4246d1fddc766cb190a04d4f984fc1ce73af3fb0 Gerrit-Change-Number: 69757 Gerrit-PatchSet: 1 Gerrit-Owner: Yen-lin Lai <yenlinlai@google.com> Gerrit-MessageType: newchange