Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#132 closed enhancement (fixed)

Please add a high-mem safe wrapper for DosQueryAppType()

Reported by: guest Owned by: bird
Priority: normal Milestone: libc-0.6.3
Component: libc Version: 0.6
Severity: normal Keywords: WinUpper DosQueryAppType
Cc: mozilla@…

Description

DosQueryAppType?() is not listed in the high-mem supporting APIs and indeed it acts weird (returncode=87 and appType=0 whatever the real appType is) when being passed a char* as first argument that is located above 512MB. So it would be great, if it a SafeDosQueryAppType?() wrapper could be written and then add it to os2safe.h.

Attachments (1)

add_SafeDosQueryAppType.diff (2.4 KB) - added by guest 17 years ago.
Patch (1st try) to implement SafeDosQueryAppType?

Download all attachments as: .zip

Change History (5)

comment:1 Changed 17 years ago by abwillis

Not sure if you would rather have a separate ticket for it but as a place to keep track of it, WinUpper? also was an unsafe call.

Changed 17 years ago by guest

Patch (1st try) to implement SafeDosQueryAppType?

comment:2 Changed 17 years ago by guest

Hmm, weird. Why does Trac only display the first part of the diff?

Anyway, again without even trying to compile libc (don't have time to even try) the diff I just attached is a first try to add a safe wrapper for DosQueryAppType?(). Don't understand what the SAFE_DOS_FAILURE() is about but copied that from other wrappers. I guess the PSCZ macros also work for PSZs...

comment:3 Changed 17 years ago by bird

Resolution: fixed
Status: newclosed

(In [3387]) Added SafeDosQueryAppType? and SafeWinUpper? (both untested). Fixes #132.

comment:4 Changed 17 years ago by bird

Keywords: WinUpper DosQueryAppType added
Milestone: libc-0.6.3
Version: 0.6
Note: See TracTickets for help on using tickets.