diff options
author | Katz Yamada <kyamada@codeaurora.org> | 2018-03-23 10:32:51 -0700 |
---|---|---|
committer | Katz Yamada <kyamada@codeaurora.org> | 2018-05-15 09:48:34 -0700 |
commit | 8bda43ca51ab1946022e9896b0f030ad7cb7d786 (patch) | |
tree | 964a0955a8e167f697ff7c3bf76ee320a43959e4 | |
parent | 80e09f69b6f186b95daf5c9200fc74ac3b8242be (diff) | |
download | gps-8bda43ca51ab1946022e9896b0f030ad7cb7d786.tar.gz |
fix: LocIpc client app unable to exit
LocIpc client apps such as garden app is unable
to delete LocIpc object since its socket listening
thread cannot be closed while it is waiting for
data and cannot be closed. Fixed to close it by
sending an abort message.
CRs-Fixed: 2213212
Change-Id: I95f26862e9faf7bd75a2f447421ba4ab7220576e
-rw-r--r-- | utils/LocIpc.cpp | 37 | ||||
-rw-r--r-- | utils/LocIpc.h | 3 |
2 files changed, 25 insertions, 15 deletions
diff --git a/utils/LocIpc.cpp b/utils/LocIpc.cpp index 26a2076..675664a 100644 --- a/utils/LocIpc.cpp +++ b/utils/LocIpc.cpp @@ -28,19 +28,9 @@ */ #include <sys/types.h> -#include <sys/socket.h> -#include <sys/un.h> #include <sys/stat.h> -#include <error.h> #include <errno.h> -#include <unistd.h> -#include <stdlib.h> -#include <array> -#include <loc_pla.h> #include <log_util.h> -#include <string> -#include <vector> -#include "gps_extended_c.h" #include "LocIpc.h" namespace loc_util { @@ -52,8 +42,10 @@ namespace loc_util { #define LOC_MSG_BUF_LEN 8192 #define LOC_MSG_HEAD "$MSGLEN$" +#define LOC_MSG_ABORT "LocIpcMsg::ABORT" class LocIpcRunnable : public LocRunnable { +friend LocIpc; public: LocIpcRunnable(LocIpc& locIpc, const std::string& ipcName) : mLocIpc(locIpc), mIpcName(ipcName) {} @@ -70,10 +62,10 @@ private: }; bool LocIpc::startListeningNonBlocking(const std::string& name) { - mRunnable.reset(new LocIpcRunnable(*this, name)); + mRunnable = new LocIpcRunnable(*this, name); std::string threadName("LocIpc-"); threadName.append(name); - return mThread.start(threadName.c_str(), mRunnable.get()); + return mThread.start(threadName.c_str(), mRunnable); } bool LocIpc::startListeningBlocking(const std::string& name) { @@ -107,6 +99,7 @@ bool LocIpc::startListeningBlocking(const std::string& name) { ssize_t nBytes = 0; std::string msg = ""; + std::string abort = LOC_MSG_ABORT; while (1) { msg.resize(LOC_MSG_BUF_LEN); nBytes = ::recvfrom(mIpcFd, (void*)(msg.data()), msg.size(), 0, NULL, NULL); @@ -116,6 +109,11 @@ bool LocIpc::startListeningBlocking(const std::string& name) { continue; } + if (strncmp(msg.data(), abort.c_str(), abort.length()) == 0) { + LOC_LOGi("recvd abort msg.data %s", msg.data()); + break; + } + if (strncmp(msg.data(), LOC_MSG_HEAD, sizeof(LOC_MSG_HEAD) - 1)) { // short message msg.resize(nBytes); @@ -142,7 +140,6 @@ bool LocIpc::startListeningBlocking(const std::string& name) { if (mStopRequested) { mStopRequested = false; return true; - } else { LOC_LOGe("cannot read socket. reason:%s", strerror(errno)); (void)::close(mIpcFd); @@ -152,14 +149,28 @@ bool LocIpc::startListeningBlocking(const std::string& name) { } void LocIpc::stopListening() { + + const char *socketName = nullptr; mStopRequested = true; + if (mRunnable) { + std::string abort = LOC_MSG_ABORT; + socketName = (reinterpret_cast<LocIpcRunnable *>(mRunnable))->mIpcName.c_str(); + send(socketName, abort); + mRunnable = nullptr; + } + if (mIpcFd >= 0) { if (::close(mIpcFd)) { LOC_LOGe("cannot close socket:%s", strerror(errno)); } mIpcFd = -1; } + + //delete from the file system at the end + if (socketName) { + unlink(socketName); + } } bool LocIpc::send(const char name[], const std::string& data) { diff --git a/utils/LocIpc.h b/utils/LocIpc.h index 0c37a7a..a1a994d 100644 --- a/utils/LocIpc.h +++ b/utils/LocIpc.h @@ -32,7 +32,6 @@ #include <string> #include <memory> -#include <string.h> #include <unistd.h> #include <sys/socket.h> #include <sys/un.h> @@ -96,7 +95,7 @@ private: int mIpcFd; bool mStopRequested; LocThread mThread; - std::unique_ptr<LocRunnable> mRunnable; + LocRunnable *mRunnable; }; class LocIpcSender { |