Skip to content

test_os debug assertion failure #83928

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

Closed
ethanhs mannequin opened this issue Feb 25, 2020 · 3 comments
Closed

test_os debug assertion failure #83928

ethanhs mannequin opened this issue Feb 25, 2020 · 3 comments
Labels
3.9 only security fixes OS-windows type-crash A hard crash of the interpreter, possibly with a core dump

Comments

@ethanhs
Copy link
Mannequin

ethanhs mannequin commented Feb 25, 2020

BPO 39747
Nosy @pfmoore, @tjguk, @zware, @zooba, @ethanhs, @iritkatriel

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2021-10-20.23:31:58.095>
created_at = <Date 2020-02-25.05:36:29.641>
labels = ['invalid', 'OS-windows', 'type-crash', '3.9']
title = 'test_os debug assertion failure'
updated_at = <Date 2021-10-20.23:31:58.095>
user = 'https://github.com/ethanhs'

bugs.python.org fields:

activity = <Date 2021-10-20.23:31:58.095>
actor = 'iritkatriel'
assignee = 'none'
closed = True
closed_date = <Date 2021-10-20.23:31:58.095>
closer = 'iritkatriel'
components = ['Windows']
creation = <Date 2020-02-25.05:36:29.641>
creator = 'ethan smith'
dependencies = []
files = []
hgrepos = []
issue_num = 39747
keywords = []
message_count = 3.0
messages = ['362624', '362630', '402226']
nosy_count = 6.0
nosy_names = ['paul.moore', 'tim.golden', 'zach.ware', 'steve.dower', 'ethan smith', 'iritkatriel']
pr_nums = []
priority = 'normal'
resolution = 'not a bug'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'crash'
url = 'https://bugs.python.org/issue39747'
versions = ['Python 3.9']

@ethanhs
Copy link
Mannequin Author

ethanhs mannequin commented Feb 25, 2020

With CPython master branch and build.bat -e -p x64, if I run test_os I get the following (in a messagebox transcribed here for ease of consumption).

Sorry if I am missing something. This means I am unable to run test_os to completion.

I am on Windows 10.0.19559.1000 x64 with CL 19.24.28315/Visual Studio 16.4.3

For test_bad_fd:
-----
Debug Assertion Failed!

Program: C:\Users\ethanhs\cpython\PCbuild\amd64\python_d.exe
File: minkernel\crts\ucrt\src\appcrt\lowio\isatty.cpp
Line: 17

Expression: (fh >= 0 && (unsigned)fh < (unsigned)_nhandle)

For information on how your program can cause an assertion
failure, see the Visual C++ documentation on asserts.

(Press Retry to debug the application)
---

@ethanhs ethanhs mannequin added 3.9 only security fixes OS-windows type-crash A hard crash of the interpreter, possibly with a core dump labels Feb 25, 2020
@zooba
Copy link
Member

zooba commented Feb 25, 2020

Well, you're running a debug build but building a release build (unless you missed the "-d" from the command line -- "-e" is the default now, BTW), so your build may be out of sync.

These assertion dialogs are supposed to be suppressed during the test suite, but only when run using "python.bat -m test test_os". If you're running the OS tests any other way, then you'll see these popups.

(I hope you used Ctrl+C in the dialog to copy the text, by the way. That's a lot to copy out by hand!)

---

FWIW, these are very good assertions for most programs. But most programs are not intentionally passing arbitrary values into system calls, so we have to suppress them for Python :)

@iritkatriel
Copy link
Member

If I understand Steve's comment it sounds like there's nothing we can do here. Shall we close it as not a bug?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.9 only security fixes OS-windows type-crash A hard crash of the interpreter, possibly with a core dump
Projects
None yet
Development

No branches or pull requests

2 participants