Crash on compare files with one panel in quick view mode

[onlyjob]

Bug-Debian: ​http://bugs.debian.org/666692

MC crashes (Segmentation fault) on "Compare files" if one of the panels in "Quick view" mode.

4.8.1, 4.8.2 and current 'master' versions are affected.

Cheers,
Dmitry.

[andrew_b]

I'm unable reproduce this.

1. Select any files on panels.
2. Switch other panel to "Quick view mode" (C-x q).
3. Compare file (C-x C-d)
4. No crash.

[onlyjob]

Could it be because you invoke 'compare directories' (C-x d) instead of 'compare files'?

Could you please try to invoke 'compare files' from menu? (Command-->[Compare files])

This is very persistent on amd64.

GNU Midnight Commander 4.8.2-32-g18df85a
Built with GLib 2.30.2
Using the S-Lang library with terminfo database
With builtin Editor
With subshell support as default
With support for background operations
With mouse support on xterm and Linux console
With support for X11 events
With internationalization support
With multiple codepages support
Virtual File Systems: cpiofs, tarfs, sfs, extfs, ext2undelfs, ftpfs, fish
Data types: char: 8; int: 32; long: 64; void *: 64; size_t: 64; off_t: 64;

'--build=x86_64-linux-gnu' '--prefix=/usr' '--includedir=${prefix}/include' '--mandir=${prefix}/share/man' '--infodir=${prefix}/share/info' '--sysconfdir=/etc' '--localstatedir=/var' '--libdir=${prefix}/lib/x86_64-linux-gnu' '--libexecdir=${prefix}/lib/x86_64-linux-gnu' '--disable-maintainer-mode' '--disable-dependency-tracking' 'AWK=awk' '--libexecdir=${prefix}/lib' '--with-x' '--with-screen=slang' '--disable-rpath' '--disable-static' '--enable-vfs-undelfs' '--enable-tests' 'build_alias=x86_64-linux-gnu' 'CFLAGS=-g -O2 -fPIE -fstack-protector --param=ssp-buffer-size=4 -Wformat -Wformat-security -Werror=format-security' 'LDFLAGS=-fPIE -pie -Wl,-z,relro -Wl,-z,now' 'CPPFLAGS=-D_FORTIFY_SOURCE=2'

[andrew_b]

Replying to onlyjob:

Could it be because you invoke 'compare directories' (C-x d) instead of 'compare files'?

No.

Could you please try to invoke 'compare files' from menu? (Command-->[Compare files])

I tried that.

This is very persistent on amd64.

Ah, I'm on i686. But it should not be dependent on arch.

[andrew_b]

Branch: 2769_compare_files_crash (parent: master).
changeset:aac197a1093c1a17d98b740027c77053cda0f22

[onlyjob]

Andrew, you're awesome - I've just tried your changeset and the problem has gone, no more crash.

Fantastic, thanks very much for rapid and professional response!

All the best,
Dmitry.

[andrew_b]

I've moved check to the upper level.

[andrew_b]

Merged to master: [d25ac6a9119a789ddf5c0d234af8a60f98aac7e8]

git log --pretty=oneline c494f7b..d25ac6a

[andrew_b]

Cherry-picked to 4.8.1-stable: [108cdbfaca45af5f320873db459e0459ee72965b]