Skip to content

bpo-45582: framework build: modPath must not be const (GH-29944) - #29944

Merged
tiran merged 1 commit into
python:mainfrom
tiran:bpo-45582-constchar
Dec 6, 2021
Merged

bpo-45582: framework build: modPath must not be const (GH-29944)#29944
tiran merged 1 commit into
python:mainfrom
tiran:bpo-45582-constchar

Conversation

@tiran

@tirantiran commented Dec 6, 2021

Copy link
Copy Markdown
Member

modPath was marked as static const although the char array is modified
later in the function. Fixes a crash when building with
--enable-framework. Issue was found by Ronald.

Co-authored-by: Ronald Oussoren ronaldoussoren@mac.com
Signed-off-by: Christian Heimes christian@python.org

https://bugs.python.org/issue45582

modPath was marked as static const although the char array is modified
later in the function. Fixes a crash when building with
--enable-framework. Issue was found by Ronald.
Co-authored-by: Ronald Oussoren <ronaldoussoren@mac.com>
Signed-off-by: Christian Heimes <christian@python.org>
@tirantiran changed the title bpo-45582: framework build: modPath must not be constbpo-45582: framework build: modPath must not be const (GH-29944)Dec 6, 2021
@tiran
tiran merged commit f16f93e into python:mainDec 6, 2021
@tiran
tiran deleted the bpo-45582-constchar branch December 6, 2021 18:13
@bedevere-bot

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot ARM64 macOS 3.x has failed when building commit f16f93e.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/725/builds/519) and take a look at the build logs.
  4. Check if the failure is related to this commit (f16f93e) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/725/builds/519

Summary of the results of the build (if available):

== Tests result: ENV CHANGED ==

407 tests OK.

10 slowest tests:

  • test_concurrent_futures: 3 min 35 sec
  • test_multiprocessing_spawn: 2 min 34 sec
  • test_multiprocessing_forkserver: 1 min 49 sec
  • test_unparse: 1 min 34 sec
  • test_asyncio: 1 min 18 sec
  • test_tokenize: 1 min 10 sec
  • test_lib2to3: 54.5 sec
  • test_logging: 53.3 sec
  • test_capi: 51.6 sec
  • test_signal: 46.2 sec

1 test altered the execution environment:
test_ftplib

17 tests skipped:
test_dbm_gnu test_devpoll test_epoll test_gdb test_ioctl
test_msilib test_multiprocessing_fork test_ossaudiodev test_spwd
test_startfile test_tix test_tk test_ttk_guionly test_winconsoleio
test_winreg test_winsound test_zipfile64

Total duration: 8 min 18 sec

Click to see traceback logs
Traceback (most recent call last):
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/test/support/_asyncore.py", line 84, in read
obj.handle_read_event()
^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/test/test_ftplib.py", line 380, in handle_read_eventself._do_ssl_handshake()
^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/test/test_ftplib.py", line 341, in _do_ssl_handshakeself.socket.do_handshake()
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/ssl.py", line 1346, in do_handshakeself._sslobj.do_handshake()
^^^^^^^^^^^^^^^^^^^^^^^^^^^ssl.SSLZeroReturnError: TLS/SSL connection has been closed (EOF) (_ssl.c:998)
Traceback (most recent call last):
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/multiprocessing/resource_tracker.py", line 209, in main
cache[rtype].remove(name)
^^^^^^^^^^^^^^^^^^^^^^^^^KeyError: '/psm_ad52fae2'
Traceback (most recent call last):
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/multiprocessing/resource_tracker.py", line 209, in main
cache[rtype].remove(name)
^^^^^^^^^^^^^^^^^^^^^^^^^KeyError: '/psm_ce0dc006'
Traceback (most recent call last):
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/threading.py", line 1031, in _bootstrap_innerself.run()
^^^^^^^^^^
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/test/test_ftplib.py", line 294, in run
asyncore.loop(timeout=0.1, count=1)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/test/support/_asyncore.py", line 208, in loop
poll_fun(timeout, map)
^^^^^^^^^^^^^^^^^^^^^^
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/test/support/_asyncore.py", line 151, in poll
read(obj)
^^^^^^^^^
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/test/support/_asyncore.py", line 88, in read
obj.handle_error()
^^^^^^^^^^^^^^^^^^
File "/Users/buildbot/buildarea/3.x.pablogsal-macos-m1.macos-with-brew/build/Lib/test/test_ftplib.py", line 417, in handle_errorraiseException^^^^^^^^^^^^^^^
Exception
k

@zoobazooba mentioned this pull request Apr 10, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tiran@bedevere-bot@the-knights-who-say-ni