This is an automated email from the ASF dual-hosted git repository.

dmeden pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 26a9de0376 CID-1508851: Possible "fix" for a false positive issue from 
coverity. (#10544)
26a9de0376 is described below

commit 26a9de03762752f99da664a72110cc7f8fa74c6d
Author: Damian Meden <dme...@apache.org>
AuthorDate: Fri Oct 6 17:18:51 2023 +0200

    CID-1508851: Possible "fix" for a false positive issue from coverity. 
(#10544)
---
 mgmt/rpc/server/IPCSocketServer.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/mgmt/rpc/server/IPCSocketServer.cc 
b/mgmt/rpc/server/IPCSocketServer.cc
index 05a07cfc16..aeb4c89825 100644
--- a/mgmt/rpc/server/IPCSocketServer.cc
+++ b/mgmt/rpc/server/IPCSocketServer.cc
@@ -218,7 +218,7 @@ IPCSocketServer::run()
     }
 
     std::error_code ec;
-    if (auto fd = this->accept(ec); !ec) {
+    if (int fd = this->accept(ec); !ec) {
       Client client{fd};
 
       if (auto [ok, errStr] = client.read_all(bw); ok) {
@@ -270,7 +270,7 @@ IPCSocketServer::create_socket(std::error_code &ec)
 int
 IPCSocketServer::accept(std::error_code &ec) const
 {
-  int ret = {-1};
+  int ret{-1};
 
   for (int retries = 0; retries < _conf.maxRetriesOnTransientErrors; 
retries++) {
     ret = ::accept(_socket, 0, 0);

Reply via email to