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

python --with-tcl-tk crashes, buffer overflow error #12114

Closed
poernahi opened this Issue Apr 5, 2017 · 15 comments

Comments

Projects
None yet
10 participants
@poernahi

poernahi commented Apr 5, 2017

Please always follow these steps:

To help us debug your issue please explain:

  • What you were trying to do (and why)
    • Trying to install python 2.7 with TK interface
    • The purpose is to run pyraf in OSX
  • What happened (include command output)
    • installed tcl-tk with brew install tcl-tk without error
    • added export PATH="/usr/local/opt/tcl-tk/bin:$PATH" to .bash_profile and sourced it
    • installed python with brew install python --with-tcl-tk without error
    • test by running python -m Tkinter => crash with Abort trap: 6
  • What you expected to happen
    • Tkinter does not crash
  • Step-by-step reproduction instructions (by running brew install commands)
    • brew install tcl-tk
    • brew install python --with-tcl-tk
    • python -m Tkinter

Additional info:

  • OSX Sierra
  • Retried after installing all OS patch and Xcode patches
  • Brew doctor reported nothing

Apple crash info:

Crashed Thread:        0  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_CRASH (SIGABRT)
Exception Codes:       0x0000000000000000, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY

Application Specific Information:
detected buffer overflow

Stack trace: --unfortunately could not figure out how to install debug symbols

frame #0: 0x00007fff90801d42 libsystem_kernel.dylib`__pthread_kill + 10
frame #1: 0x00007fff908ef5bf libsystem_pthread.dylib`pthread_kill + 90
frame #2: 0x00007fff90767420 libsystem_c.dylib`abort + 129
frame #3: 0x00007fff90767592 libsystem_c.dylib`abort_report_np + 181
frame #4: 0x00007fff9078df28 libsystem_c.dylib`__chk_fail + 48
frame #5: 0x00007fff9078def8 libsystem_c.dylib`__chk_fail_overflow + 16
frame #6: 0x00007fff9078e145 libsystem_c.dylib`__strcpy_chk + 83
frame #7: 0x00000001025cbe86 libtk8.6.dylib`Tk_WmObjCmd + 12920
frame #8: 0x000000010265779f libtcl8.6.dylib`TclNRRunCallbacks + 80
frame #9: 0x00000001005eea92 _tkinter.so`Tkapp_Call + 425
frame #10: 0x0000000100116623 Python`PyEval_EvalFrameEx + 9903
frame #11: 0x0000000100113d7f Python`PyEval_EvalCodeEx + 1621
frame #12: 0x000000010011c58d Python`fast_function + 298
frame #13: 0x0000000100116557 Python`PyEval_EvalFrameEx + 9699
frame #14: 0x000000010011c528 Python`fast_function + 197
--snip--
frame #37: 0x00007fff906d3235 libdyld.dylib`start + 1
frame #38: 0x00007fff906d3235 libdyld.dylib`start + 1
@poernahi

This comment has been minimized.

Show comment
Hide comment
@poernahi

poernahi Apr 6, 2017

Issue went away after brewing python without --with-tcl-tk

poernahi commented Apr 6, 2017

Issue went away after brewing python without --with-tcl-tk

@poernahi poernahi closed this Apr 6, 2017

@jkl1337 jkl1337 referenced this issue Apr 24, 2017

Closed

SIGABRT with python3 --with-tcl-tk #12808

4 of 4 tasks complete
@schiotz

This comment has been minimized.

Show comment
Hide comment
@schiotz

schiotz May 4, 2017

@poernahi
Could you please reopen this issue, as it is a real issue. The version of Tcl/Tk provided by Apple is slow and buggy (https://www.python.org/download/mac/tcltk/), using the homebrew-built version should be better. But of course not when it crashes.

schiotz commented May 4, 2017

@poernahi
Could you please reopen this issue, as it is a real issue. The version of Tcl/Tk provided by Apple is slow and buggy (https://www.python.org/download/mac/tcltk/), using the homebrew-built version should be better. But of course not when it crashes.

@poernahi poernahi reopened this May 8, 2017

@poernahi

This comment has been minimized.

Show comment
Hide comment
@poernahi

poernahi May 8, 2017

Reopening as requested.

poernahi commented May 8, 2017

Reopening as requested.

@stale stale bot added the stale label May 29, 2017

@stale

This comment has been minimized.

Show comment
Hide comment
@stale

stale bot May 29, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale bot commented May 29, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@iMichka

This comment has been minimized.

Show comment
Hide comment
@iMichka

iMichka May 30, 2017

Contributor

Can also be reproduced with:

python
import Tkinter
Tkinter._test()

@tdsmith I think this is what is making pymol crash in homebrew-science

Contributor

iMichka commented May 30, 2017

Can also be reproduced with:

python
import Tkinter
Tkinter._test()

@tdsmith I think this is what is making pymol crash in homebrew-science

@stale stale bot removed the stale label May 30, 2017

@rbf22

This comment has been minimized.

Show comment
Hide comment
@rbf22

rbf22 Jun 8, 2017

Is anyone working on this?

rbf22 commented Jun 8, 2017

Is anyone working on this?

@m-yac

This comment has been minimized.

Show comment
Hide comment
@m-yac

m-yac Jun 8, 2017

Seconded – this is still very much an issue. I also get this error with iMichka's minimal example.

m-yac commented Jun 8, 2017

Seconded – this is still very much an issue. I also get this error with iMichka's minimal example.

@MikeMcQuaid

This comment has been minimized.

Show comment
Hide comment
@MikeMcQuaid

MikeMcQuaid Jun 9, 2017

Member

Is anyone working on this?

No. Someone experiencing the issue will need to.

Member

MikeMcQuaid commented Jun 9, 2017

Is anyone working on this?

No. Someone experiencing the issue will need to.

@cowsandmilk

This comment has been minimized.

Show comment
Hide comment
@cowsandmilk

cowsandmilk Jun 22, 2017

Contributor

bringing back the --with-x11 removed in 1b5d094 fixes this for iMichka's example (and pymol as well, which is my main concern).

What was the justification for removing that option? The commit references something seemingly irrelevant to me.

Contributor

cowsandmilk commented Jun 22, 2017

bringing back the --with-x11 removed in 1b5d094 fixes this for iMichka's example (and pymol as well, which is my main concern).

What was the justification for removing that option? The commit references something seemingly irrelevant to me.

@ilovezfs

This comment has been minimized.

Show comment
Hide comment
@ilovezfs

ilovezfs Jun 27, 2017

Contributor

Python 3 has the same bug: #12808.

Contributor

ilovezfs commented Jun 27, 2017

Python 3 has the same bug: #12808.

@stale stale bot added the stale label Jul 19, 2017

@stale

This comment has been minimized.

Show comment
Hide comment
@stale

stale bot Jul 19, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale bot commented Jul 19, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

telamonian pushed a commit to telamonian/homebrew-core that referenced this issue Jul 22, 2017

telamonian
tcl-tk: fix for Homebrew#12114 and Homebrew#12808. Reenabled the `--w…
…ith-x11` option. Without --with-x11 `brew test python` and `brew test python3` fail, and some formulae (eg pymol) break
@telamonian

This comment has been minimized.

Show comment
Hide comment
@telamonian

telamonian Jul 22, 2017

Contributor

@cowsandmilk's suggestion to readd (and use) the --with-x11 option in the tcl-tk formula fixed these issues for me. I've made a pull request with the revised formula. This apparently also fixes #12808.

Contributor

telamonian commented Jul 22, 2017

@cowsandmilk's suggestion to readd (and use) the --with-x11 option in the tcl-tk formula fixed these issues for me. I've made a pull request with the revised formula. This apparently also fixes #12808.

@stale stale bot removed the stale label Jul 22, 2017

@ilovezfs

This comment has been minimized.

Show comment
Hide comment
@ilovezfs

ilovezfs Jul 22, 2017

Contributor

I've closed that PR but an actual fix for these issues would be most welcome.

Contributor

ilovezfs commented Jul 22, 2017

I've closed that PR but an actual fix for these issues would be most welcome.

BrewTestBot added a commit to BrewTestBot/homebrew-core that referenced this issue Jul 22, 2017

tcl-tk: fix for Homebrew#12114 and Homebrew#12808. Reenabled the `--w…
…ith-x11` option. Without --with-x11 `brew test python` and `brew test python3` fail, and some formulae (eg pymol) break

telamonian pushed a commit to telamonian/homebrew-core that referenced this issue Jul 23, 2017

telamonian
tcl-tk: fixes Homebrew#12114 for python and Homebrew#12808 for python…
…3 by bumping tcl-tk

to a version in which the upstream has fixed the relevant bugs

BrewTestBot added a commit to BrewTestBot/homebrew-core that referenced this issue Jul 23, 2017

tcl-tk: fixes Homebrew#12114 for python and Homebrew#12808 for python…
…3 by bumping tcl-tk to a version in which the upstream has fixed the relevant bugs

BrewTestBot added a commit to BrewTestBot/homebrew-core that referenced this issue Jul 23, 2017

tcl-tk: fixes Homebrew#12114 for python and Homebrew#12808 for python…
…3 by bumping tcl-tk to a version in which the upstream has fixed the relevant bugs

telamonian pushed a commit to telamonian/homebrew-core that referenced this issue Jul 25, 2017

telamonian pushed a commit to telamonian/homebrew-core that referenced this issue Aug 1, 2017

telamonian
tcl-tk: fix python tkinter issues Homebrew#12114 and Homebrew#12808 b…
…y adding devel

build. Also removes semi-function HEAD build

BrewTestBot added a commit to BrewTestBot/homebrew-core that referenced this issue Aug 1, 2017

BrewTestBot added a commit to BrewTestBot/homebrew-core that referenced this issue Aug 1, 2017

tcl-tk: fix python tkinter issues Homebrew#12114 and Homebrew#12808 b…
…y adding devel build. Also removes semi-function HEAD build

BrewTestBot added a commit to BrewTestBot/homebrew-core that referenced this issue Aug 1, 2017

BrewTestBot added a commit to BrewTestBot/homebrew-core that referenced this issue Aug 1, 2017

tcl-tk: fix python tkinter issues Homebrew#12114 and Homebrew#12808 b…
…y adding devel build. Also removes semi-function HEAD build

@ilovezfs ilovezfs closed this in 4a61e09 Aug 2, 2017

@ilovezfs

This comment has been minimized.

Show comment
Hide comment
@ilovezfs

ilovezfs Aug 2, 2017

Contributor

This should be fixed if you

brew install --devel tcl-tk
Contributor

ilovezfs commented Aug 2, 2017

This should be fixed if you

brew install --devel tcl-tk

javian pushed a commit to javian/homebrew-core that referenced this issue Aug 10, 2017

@MikeTam1021

This comment has been minimized.

Show comment
Hide comment
@MikeTam1021

MikeTam1021 Sep 11, 2017

this did not work for me ilovezfs

MikeTam1021 commented Sep 11, 2017

this did not work for me ilovezfs

@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.