Ticket #4598 (closed defect: fixed)

Opened 2 months ago

Last modified 2 months ago

Midnight Commander Git (Today) Tar component segfaults

Reported by: Axanar Owned by: andrew_b
Priority: major Milestone: 4.8.33
Component: mc-vfs Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description

Hi there.

I've been using the Git version of Midnight Commander for a few days, after we got the TAR compilation issue fixed - due the TAR update.

Although. The VFS System tends to segfault now. Sometimes it works, sometimes it won't. I roughly testet this with a bunch of files now.

Fedora 41

The TAR file that causes segfaults here can be found here:

​https://freedesktop.org/software/pulseaudio/pavucontrol/
​http://freedesktop.org/software/pulseaudio/pavucontrol/pavucontrol-6.1.tar.gz

Start MC, select and click on the file and it segfaults.

Normal tar cfzv works perfectly on that file.

Any thoughts?

Attachments

log.txt​ (10.3 KB) - added by Axanar 2 months ago.
Full backtrace of captured coredump

Change History

Changed 2 months ago by Axanar

Full backtrace of captured coredump

comment:1 Changed 2 months ago by andrew_b

  • Status changed from new to accepted
  • Owner set to andrew_b
  • Component changed from mc-core to mc-vfs
  • Milestone changed from Future Releases to 4.8.33

Bad changeset:7d8deb89c892f79892526706b421b8680ea26cef.

comment:2 follow-up: ↓ 3 Changed 2 months ago by andrew_b

  • Branch state changed from no branch to on review

Thanks for testing!

Branch: 4598_tar_segfault
Initial changeset:d1863c00618987f736373ac00b57908f910663cd

comment:3 in reply to: ↑ 2 Changed 2 months ago by Axanar

Replying to andrew_b:

Thanks for testing!

Branch: 4598_tar_segfault
Initial changeset:d1863c00618987f736373ac00b57908f910663cd

I've tested the Branch with a bunch of tar files and so far everything works!

Thanks!

comment:4 Changed 2 months ago by zaytsev

  • Votes for changeset set to zaytsev
  • Branch state changed from on review to approved

comment:5 Changed 2 months ago by andrew_b

  • Status changed from accepted to testing
  • Votes for changeset changed from zaytsev to committed-master
  • Resolution set to fixed
  • Branch state changed from approved to merged

Merged to master: [54a4c7d55f8aa07e57886f50c194653ec133a088].

git log --oneline b294d650a..54a4c7d55
Last edited 2 months ago by andrew_b (previous) (diff)

comment:6 Changed 2 months ago by andrew_b

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.