[SRU] segfaults with certain trace.dat files

debdiff attached.

Patch looks good. One minor nitpick was that update-maintainer needed running to correct the maintainer address in d/control, but I can trivially do that. Sponsoring for lunar.

On Sun, 5 Nov 2023 at 12:20, Dave Jones <email address hidden> wrote:
>
> Patch looks good. One minor nitpick was that update-maintainer needed
> running to correct the maintainer address in d/control, but I can
> trivially do that. Sponsoring for lunar.

Thanks for sponsoring Dave,

But a complete newbie question as I am still learning how Ubuntu works.
Why does the "maintainer address" need updating? I am the Debian
maintainer of this package.

--
Regards
Sudip

UPDATE: Thanks Dave for explaining why the maintainer address needed updating.

Why was the Debian portion of the version bumped? it's 2.2.0-2ubuntu0.1 while it should have been -1ubuntu0.1 IMO

Thanks for rejecting that Timo and sorry for that.

Modified debdiff attached now.

Hello Sudip, or anyone else affected,

Accepted kernelshark into lunar-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/kernelshark/2.2.0-1ubuntu0.23.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-lunar to verification-done-lunar. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-lunar. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

well, since mantic and up already had a newer version, the lunar upload could just have used -1u0.1, but that's bikeshedding

I can confirm that the kernelshark package in lunar-proposed has fixed the bug for me.

Test done:
1. Install kernelshark on a Lunar installation.
2. Download the trace.dat file from https://bugzilla.kernel.org/show_bug.cgi?id=217429
3. Open the trace with kernelshark and confirm that it is resulting in a segfault.
4. Add lunar-proposed to apt sources
5. Update kernelshark from lunar-proposed.
6. Open the same downloaded trace with kernelshark.

Test result: kernelshark was able to open the trace.

Package tested:
$ dpkg -l | grep kernelshark
ii kernelshark 2.9.3+really2.2.0-1ubuntu0.23.04.1 amd64 Utility for retrieving and analyzing function tracing in the kernel

This bug was fixed in the package kernelshark - 2.2.0-1ubuntu0.23.04.1

---------------
kernelshark (2.2.0-1ubuntu0.23.04.1) lunar; urgency=medium

  * Add upstream patch to fix segfault. (LP: #2042720)

 -- Sudip Mukherjee <email address hidden> Fri, 17 Nov 2023 22:34:52 +0000

The verification of the Stable Release Update for kernelshark has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.