Ticket #4244 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Password with segmentation fault

Reported by: JonasBelsewir Owned by: andrew_b
Priority: major Milestone: 4.8.27
Component: mc-core Version: 4.8.26
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description (last modified by andrew_b) (diff)

Hi,

1.
Debian 11 Testing (Xfce4 4.16)
I open a terminal window
I start: mc ftp://user@192.168.1.1/san32
mc opens and asks me for the password,
but there is a segmentation fault,
see attached image.

2.
I do not get this error with
version 4.8.25 and older.

Thanks
Jonas

LC_MESSAGES=C mc -V
GNU Midnight Commander 4.8.26
Built with GLib 2.66.4
Built with S-Lang 2.3.2 with terminfo database
With builtin Editor and Aspell support
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
With ext2fs attributes support
Virtual File Systems:
 cpiofs, tarfs, sfs, extfs, ext2undelfs, ftpfs, sftpfs, fish
Data types:
 char: 8; int: 32; long: 64; void *: 64; size_t: 64; off_t: 64;
LC_MESSAGES=C mc -F
Home directory: /home/user
Profile root directory: /home/user

[System data]
    Config directory: /etc/mc/
    Data directory:   /usr/share/mc/
    File extension handlers: /usr/lib/mc/ext.d/
    VFS plugins and scripts: /usr/lib/mc/
	extfs.d:        /usr/lib/mc/extfs.d/
	fish:           /usr/lib/mc/fish/

[User data]
    Config directory: /home/user/.config/mc/
    Data directory:   /home/user/.local/share/mc/
	skins:          /home/user/.local/share/mc/skins/
	extfs.d:        /home/user/.local/share/mc/extfs.d/
	fish:           /home/user/.local/share/mc/fish/
	mcedit macros:  /home/user/.local/share/mc/mc.macros
	mcedit external macros: /home/user/.local/share/mc/mcedit/macros.d/macro.*
    Cache directory:  /home/user/.cache/mc/
mc --configure-options
 '--build=x86_64-linux-gnu' '--prefix=/usr' '--includedir=${prefix}/include' '--mandir=${prefix}/share/man' '--infodir=${prefix}/share/info' '--sysconfdir=/etc' '--localstatedir=/var' '--disable-option-checking' '--libdir=${prefix}/lib/x86_64-linux-gnu' '--runstatedir=/run' '--disable-maintainer-mode' '--disable-dependency-tracking' 'AWK=awk' 'X11_WWW=x-www-browser' '--libexecdir=/usr/lib' '--with-x' '--with-screen=slang' '--disable-rpath' '--disable-static' '--disable-silent-rules' '--enable-aspell' '--enable-vfs-sftp' '--enable-vfs-undelfs' '--enable-tests' 'build_alias=x86_64-linux-gnu' 'CFLAGS=-g -O2 -ffile-prefix-map=/build/mc-FTvHQk/mc-4.8.26=. -fstack-protector-strong -Wformat -Werror=format-security' 'LDFLAGS=-Wl,-z,relro -Wl,-z,now -Wl,--as-needed' 'CPPFLAGS=-Wdate-time -D_FORTIFY_SOURCE=2'
user@pc2:~$ 

Attachments

mc-segfault.png (22.5 KB) - added by JonasBelsewir 3 years ago.

Change History

Changed 3 years ago by JonasBelsewir

comment:1 Changed 3 years ago by andrew_b

  • Version changed from master to 4.8.26
  • Description modified (diff)

comment:2 in reply to: ↑ description Changed 3 years ago by andrew_b

  • Owner set to andrew_b
  • Status changed from new to accepted
  • Branch state changed from no branch to on review
  • Milestone changed from Future Releases to 4.8.27

Replying to JonasBelsewir:

I do not get this error with
version 4.8.25 and older.

Yes. This is yet another effect of #4114.

Branch: 4244_dialog_segfault
changeset:0375997c789eef9b6355aadd80aaf4aced68c560

comment:3 Changed 3 years ago by JonasBelsewir

I downloaded code (mc-4244...) from git and compiled myself.

It works OK now.
Thank you very much for the fix.

Last edited 3 years ago by JonasBelsewir (previous) (diff)

comment:4 Changed 3 years ago by andrew_b

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

comment:5 Changed 3 years ago by andrew_b

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

comment:6 Changed 3 years ago by andrew_b

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