Bug 467816 - Coredump on TimeoutChecker::pauseTimeoutCheck when getitimer(, NULL, ), on 32bit _TIME_BITS=64
Summary: Coredump on TimeoutChecker::pauseTimeoutCheck when getitimer(, NULL, ), on 3...
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kjs
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.96.0
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Bernd Buschinski
URL:
Keywords:
: 467817 (view as bug list)
Depends on:
Blocks:
 
Reported: 2023-03-26 15:01 UTC by Taketo Kabe
Modified: 2023-07-08 18:37 UTC (History)
2 users (show)

See Also:
Latest Commit: https://invent.kde.org/frameworks/kjs/-/commit/cf19cfe8fcbe1c2d651c256a6d0898fcf0576d13
Version Fixed In:

Attachments
Patch t fix setitimer( , NULL, ) (570 bytes, message/rfc822)
2023-03-26 15:01 UTC, Taketo Kabe 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Taketo Kabe 2023-03-26 15:01:17 UTC 
Created attachment 157590 [details]
Patch t fix setitimer( , NULL, )

Inputtinhg text in Google search box crashed konqueror.

STEPS TO REPRODUCE
1.  Prepare konquer with kf5-khtml enabled
2.  Nagivate http://www.google.co.jp/
3. Input some text USING some impot method

OBSERVED RESULT
konqueror coredump

EXPECTED RESULT
ko coredump

SOFTWARE/OS VERSIONS
Linux/KDE Plasma:  5.14.0-162.6.1.el9_1.0.1.x86_64 #1 SMP PREEMPT_DYNAMIC Mon Nov 28 18:44:09 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
KDE Plasma Version:  5.24.6
KDE Frameworks Version:  5.96.0
Qt Version:  5.15.3

ADDITIONAL INFORMATION
Passing NULL to 2nd argument in
int setitimer(int which, const struct itimerval *new_value, struct itimerval *old_value)
is not supported. Kernel emits warning, and
glibc on 32bit with _TIME_BITS=64 will coredump immediatedy due to 
deferecencing new_value->it_interval.

The attached patch seems to fix this.
Comment 1 Christoph Cullmann 2023-07-03 20:21:18 UTC 
*** Bug 467817 has been marked as a duplicate of this bug. ***
Comment 2 Christoph Cullmann 2023-07-03 20:22:00 UTC 
Could you submit that as patch to the kf5 branch here:

https://invent.kde.org/frameworks/kjs/-/merge_requests
Comment 3 Taketo Kabe 2023-07-08 17:59:42 UTC 
FIxed in merge https://invent.kde.org/frameworks/kjs/-/merge_requests/10
Comment 4 Christoph Cullmann 2023-07-08 18:37:58 UTC 
Thanks for helping to get that fix in.