Skip to content

Commit 857eff8

Browse files
authored
Merge pull request rdkcentral#6304 from asurdej-comcast/sighup_main
RDKTV-34129: [WebKitBrowser] Use SIGHUP for suspended web process
2 parents df402c6 + 94e7d1e commit 857eff8

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed

WebKitBrowser/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,10 @@ All notable changes to this RDK Service will be documented in this file.
1717
* For more details, refer to [versioning](https://github.com/rdkcentral/rdkservices#versioning) section under Main README.
1818

1919

20+
## [1.1.30] - 2025-06-27
21+
### Fixed
22+
- Change kill signal for unresponsive suspended WebProcess
23+
2024
## [1.1.29] - 2025-06-05
2125
### Fixed
2226
- Fixed Numeric Truncation Error

WebKitBrowser/WebKitBrowser.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121

2222
#define API_VERSION_NUMBER_MAJOR 1
2323
#define API_VERSION_NUMBER_MINOR 1
24-
#define API_VERSION_NUMBER_PATCH 29
24+
#define API_VERSION_NUMBER_PATCH 30
2525

2626
namespace WPEFramework {
2727

WebKitBrowser/WebKitImplementation.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3531,8 +3531,9 @@ static GSourceFuncs _handlerIntervention =
35313531
if (_unresponsiveReplyNum <= kWebProcessUnresponsiveReplyDefaultLimit) {
35323532
_unresponsiveReplyNum = kWebProcessUnresponsiveReplyDefaultLimit;
35333533
Logging::DumpSystemFiles(webprocessPID);
3534-
if (syscall(__NR_tgkill, webprocessPID, webprocessPID, SIGFPE) == -1) {
3535-
SYSLOG(Logging::Error, (_T("tgkill failed, signal=%d process=%u errno=%d (%s)"), SIGFPE, webprocessPID, errno, strerror(errno)));
3534+
// Kill with SIGHUP with no coredump/minidump
3535+
if (syscall(__NR_tgkill, webprocessPID, webprocessPID, SIGHUP) == -1) {
3536+
SYSLOG(Logging::Error, (_T("tgkill failed, signal=%d process=%u errno=%d (%s)"), SIGHUP, webprocessPID, errno, strerror(errno)));
35363537
}
35373538
} else {
35383539
DeactivateBrowser(PluginHost::IShell::FAILURE);

0 commit comments

Comments
 (0)