Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

crash in copy progress dialog #2448

Closed
mc-butler opened this issue Dec 13, 2010 · 9 comments
Closed

crash in copy progress dialog #2448

mc-butler opened this issue Dec 13, 2010 · 9 comments
Assignees
Labels
area: core Issues not related to a specific subsystem prio: high Serious problem that could block progress
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/2448
Reporter ossi (@ossilator)

i guess this goes without comment:

Program terminated with signal 8, Arithmetic exception.
#0  0x080e7dca in __udivdi3 ()
#1  0x080a3487 in copy_file_file_display_progress (tctx=0x91e2300, ctx=0x91ee078,
    src_path=0x921ab00 "/home/obuddenh/maildir-troll/qt-preview-feedback/.mbsyncstate.journal",
    dst_path=0x91ef638 "./maildir-troll-backup/qt-preview-feedback/.mbsyncstate.journal") at file.c:532
#2  copy_file_file (tctx=0x91e2300, ctx=0x91ee078,
    src_path=0x921ab00 "/home/obuddenh/maildir-troll/qt-preview-feedback/.mbsyncstate.journal",
    dst_path=0x91ef638 "./maildir-troll-backup/qt-preview-feedback/.mbsyncstate.journal") at file.c:1500
#3  0x080a3d55 in copy_dir_dir (tctx=0x91e2300, ctx=0x91ee078, s=0x91e79d8 "/home/obuddenh/maildir-troll/qt-preview-feedback",
    _d=0x91e9300 "./maildir-troll-backup/qt-preview-feedback", toplevel=0, move_over=0, do_delete=0, parent_dirs=0x92093c8)
    at file.c:1805
#4  0x080a3e0a in copy_dir_dir (tctx=0x91e2300, ctx=0x91ee078, s=0x921b030 "/home/obuddenh/maildir-troll",
    _d=0x91f4068 "./maildir-troll-backup", toplevel=1, move_over=0, do_delete=0, parent_dirs=0x0) at file.c:1797
#5  0x080a567e in panel_operate (source_panel=0x91fb458, operation=OP_COPY, force_single=1) at file.c:2480
#6  0x0809a197 in copy_cmd_local () at cmd.c:839
#7  0x080bb62a in panel_execute_cmd (panel=0x91fb458, command=<value optimized out>) at panel.c:2745
#8  0x08061ec7 in send_message (h=0x91ed378, key=1015, event=0xffaa1660) at ../../lib/widget/widget-common.h:132
#9  dlg_key_event (h=0x91ed378, key=1015, event=0xffaa1660) at dialog.c:476
#10 dlg_process_event (h=0x91ed378, key=1015, event=0xffaa1660) at dialog.c:1113
#11 0x0806247a in frontend_run_dlg (h=0x91ed378) at dialog.c:529
#12 run_dlg (h=0x91ed378) at dialog.c:1145
#13 0x080b379e in create_panels_and_run_mc () at midnight.c:877
#14 do_nc () at midnight.c:1600
#15 0x0805714b in main (argc=3, argv=0xffaa18d4) at main.c:527
@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 13, 2010 at 10:14 UTC (comment 1)

Is verbose off?

@mc-butler
Copy link
Author

Changed by ossi (@ossilator) on Dec 13, 2010 at 10:25 UTC (comment 2)

no

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 13, 2010 at 11:30 UTC (comment 3)

  • Severity changed from no branch to on review
  • Owner set to andrew_b
  • Status changed from new to accepted
  • Milestone changed from 4.8 to 4.7.5

Hm, I can't reproduce this division by zero.

Created 2448_copy_progress_crash branch.
Initial [fc80aac0450dc703154c9f8029963a9b64110200]\

@mc-butler
Copy link
Author

Changed by ossi (@ossilator) on Dec 13, 2010 at 11:50 UTC (comment 4)

the reason is probably that the first copied file deep in the tree is empty.

the patch looks hacky, but i don't know the surrounding code. will it show something like "---" when the ETA is unreasonably large?

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 13, 2010 at 13:28 UTC (comment 4.5)

Replying to ossi:

will it show something like "---" when the ETA is unreasonably large?

I don't know. :)

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 14, 2010 at 13:08 UTC (comment 6)

  • Votes set to slavazanko

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Dec 23, 2010 at 10:59 UTC (comment 7)

  • Severity changed from on review to approved
  • Votes changed from slavazanko to slavazanko angel_il

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 23, 2010 at 11:25 UTC (comment 8)

  • Resolution set to fixed
  • Votes changed from slavazanko angel_il to committed-master
  • Severity changed from approved to merged
  • Status changed from accepted to testing

Merged to master.
[a557605]

git log --pretty=oneline 31d440c..a557605

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 23, 2010 at 11:26 UTC (comment 9)

  • Status changed from testing to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues not related to a specific subsystem prio: high Serious problem that could block progress
Development

No branches or pull requests

2 participants