Ros-diffs

ros-diffs@reactos.org

81390 discussions

[gbrunmar] 31804: Added missing decleration of StringCbPrintfA and StringCbPrintfW to strsafe.h.
by gbrunmar＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: gbrunmar Date: Mon Jan 14 21:38:48 2008 New Revision: 31804 URL: http://svn.reactos.org/svn/reactos?rev=31804&view=rev Log: Added missing decleration of StringCbPrintfA and StringCbPrintfW to strsafe.h. Modified: trunk/reactos/include/psdk/strsafe.h Modified: trunk/reactos/include/psdk/strsafe.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/strsafe.h?rev… ============================================================================== --- trunk/reactos/include/psdk/strsafe.h (original) +++ trunk/reactos/include/psdk/strsafe.h Mon Jan 14 21:38:48 2008 @@ -342,6 +342,18 @@ unsigned long dwFlags); STRSAFEAPI +StringCbPrintfA(LPTSTR pszDest, + size_t cbDest, + LPCTSTR pszFormat, + ...); + +STRSAFEAPI +StringCbPrintfW(LPWSTR pszDest, + size_t cbDest, + LPCWSTR pszFormat, + ...); + +STRSAFEAPI StringCbPrintfExA(LPTSTR pszDest, size_t cbDest, LPTSTR *ppszDestEnd,

1 0

[akorotaev] 31803: Fix visual glitch in usetup on BootLoader page when pressing up key
by akorotaev＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: akorotaev Date: Mon Jan 14 21:05:28 2008 New Revision: 31803 URL: http://svn.reactos.org/svn/reactos?rev=31803&view=rev Log: Fix visual glitch in usetup on BootLoader page when pressing up key Modified: trunk/reactos/base/setup/usetup/interface/usetup.c Modified: trunk/reactos/base/setup/usetup/interface/usetup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/setup/usetup/interfac… ============================================================================== --- trunk/reactos/base/setup/usetup/interface/usetup.c (original) +++ trunk/reactos/base/setup/usetup/interface/usetup.c Mon Jan 14 21:05:28 2008 @@ -3270,13 +3270,13 @@ else if ((Ir->Event.KeyEvent.uChar.AsciiChar == 0x00) && (Ir->Event.KeyEvent.wVirtualKeyCode == VK_UP)) /* UP */ { - CONSOLE_NormalTextXY (8, Line, 48, 1); + CONSOLE_NormalTextXY (8, Line, 60, 1); Line--; if (Line<12) Line=14; if (Line>14) Line=12; - CONSOLE_InvertTextXY (8, Line, 48, 1); + CONSOLE_InvertTextXY (8, Line, 60, 1); } else if ((Ir->Event.KeyEvent.uChar.AsciiChar == 0x00) && (Ir->Event.KeyEvent.wVirtualKeyCode == VK_F3)) /* F3 */

1 0

[fireball] 31802: - Fix a bug (% is a reserved character in string operations).
by fireball＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: fireball Date: Mon Jan 14 20:26:46 2008 New Revision: 31802 URL: http://svn.reactos.org/svn/reactos?rev=31802&view=rev Log: - Fix a bug (% is a reserved character in string operations). Modified: trunk/reactos/base/system/services/rpcserver.c Modified: trunk/reactos/base/system/services/rpcserver.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/services/rpcse… ============================================================================== --- trunk/reactos/base/system/services/rpcserver.c (original) +++ trunk/reactos/base/system/services/rpcserver.c Mon Jan 14 20:26:46 2008 @@ -1279,7 +1279,7 @@ /* Check if it has %SystemRoot% (len=13) */ if (ServiceNameLen > 13 && - !wcsnicmp(L"%SystemRoot%\\", lpServiceName, 13)) + !wcsnicmp(L"%%SystemRoot%%\\", lpServiceName, 13)) { /* Substitute %SystemRoot% with \\SystemRoot\\ */ *lpCanonName = LocalAlloc(LMEM_ZEROINIT, ServiceNameLen * sizeof(WCHAR) + sizeof(WCHAR));

1 0

[fireball] 31801: - Add special handling of drivers and win32 services binarynames. A few buffer overflows fixed by Herve. See issue #2962 for more details.
by fireball＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: fireball Date: Mon Jan 14 20:15:19 2008 New Revision: 31801 URL: http://svn.reactos.org/svn/reactos?rev=31801&view=rev Log: - Add special handling of drivers and win32 services binarynames. A few buffer overflows fixed by Herve. See issue #2962 for more details. Modified: trunk/reactos/dll/win32/setupapi/install.c Modified: trunk/reactos/dll/win32/setupapi/install.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/setupapi/install… ============================================================================== --- trunk/reactos/dll/win32/setupapi/install.c (original) +++ trunk/reactos/dll/win32/setupapi/install.c Mon Jan 14 20:15:19 2008 @@ -1619,6 +1619,49 @@ return ret; } +static VOID FixupServiceBinaryPath( + IN DWORD ServiceType, + IN OUT LPWSTR *ServiceBinary) +{ + LPWSTR Buffer; + WCHAR ReactosDir[MAX_PATH]; + DWORD RosDirLength, ServiceLength, Win32Length; + + GetWindowsDirectoryW(ReactosDir, MAX_PATH); + RosDirLength = strlenW(ReactosDir); + ServiceLength = strlenW(*ServiceBinary); + + /* Check and fix two things: + 1. Get rid of C:\ReactOS and use relative + path instead. + 2. Add %SystemRoot% for Win32 services */ + + if (ServiceLength < RosDirLength) + return; + + if (!wcsnicmp(*ServiceBinary, ReactosDir, RosDirLength)) + { + /* Yes, the first part is the C:\ReactOS\, just skip it */ + MoveMemory(*ServiceBinary, *ServiceBinary + RosDirLength + 1, + (ServiceLength - RosDirLength) * sizeof(WCHAR)); + + /* Handle Win32-services differently */ + if (ServiceType & SERVICE_WIN32) + { + Win32Length = (ServiceLength - + RosDirLength - 1 + 13) * sizeof(WCHAR); + /* -1 to not count the separator after C:\ReactOS + wcslen(L"%SystemRoot%\\") = 13*sizeof(wchar_t) */ + Buffer = MyMalloc(Win32Length); + + wcscpy(Buffer, L"%%SystemRoot%%\\"); + wcscat(Buffer, *ServiceBinary); + MyFree(*ServiceBinary); + + *ServiceBinary = Buffer; + } + } +} static BOOL InstallOneService( struct DeviceInfoSet *list, @@ -1662,6 +1705,9 @@ if (!GetLineText(hInf, ServiceSection, ServiceBinaryKey, &ServiceBinary)) goto cleanup; + + /* Adjust binary path according to the service type */ + FixupServiceBinaryPath(ServiceType, &ServiceBinary); /* Don't check return value, as these fields are optional and * GetLineText initialize output parameter even on failure */

1 0

[hpoussin] 31800: Update status
by hpoussin＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: hpoussin Date: Mon Jan 14 20:08:42 2008 New Revision: 31800 URL: http://svn.reactos.org/svn/reactos?rev=31800&view=rev Log: Update status Modified: trunk/reactos/media/doc/README.WINE Modified: trunk/reactos/media/doc/README.WINE URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/doc/README.WINE?rev=… ============================================================================== --- trunk/reactos/media/doc/README.WINE (original) +++ trunk/reactos/media/doc/README.WINE Mon Jan 14 20:08:42 2008 @@ -33,7 +33,7 @@ reactos/dll/win32/advpack # Autosync reactos/dll/win32/avifil32 # Autosync -reactos/dll/win32/browseui # Autosync +reactos/dll/win32/browseui # Out of sync reactos/dll/win32/cabinet # Autosync reactos/dll/win32/clusapi # Autosync reactos/dll/win32/comcat # Autosync @@ -55,17 +55,17 @@ reactos/dll/win32/mpr # Autosync reactos/dll/win32/msacm32 # Out of sync reactos/dll/win32/msimg32 # Autosync -reactos/dll/win32/msi # Needs a widl update -reactos/dll/win32/msvcrt20 # Out of sync +reactos/dll/win32/msi # Needs a sync script/rbuild update +reactos/dll/win32/msvcrt20 # Autosync reactos/dll/win32/msvfw32 # Autosync reactos/dll/win32/msxml3 # Synced to Wine-20071230 reactos/dll/win32/netapi32 # Autosync reactos/dll/win32/objsel # Autosync reactos/dll/win32/odbc32 # Out of sync. Depends on port of Linux ODBC. reactos/dll/win32/odbccp32 # Autosync -reactos/dll/win32/ole32 # Needs a widl update +reactos/dll/win32/ole32 # Autosync reactos/dll/win32/oleacc # Autosync -reactos/dll/win32/oleaut32 # Needs a widl update +reactos/dll/win32/oleaut32 # Needs a sync script/rbuild update reactos/dll/win32/oledlg # Autosync reactos/dll/win32/olepro32 # Autosync reactos/dll/win32/powrprof # Autosync @@ -109,7 +109,7 @@ from Winehq CVS. If you are looking to update something in these files check Wine current souces first as it may already be fixed. -reactos/lib/uuid # +reactos/lib/uuid # Synced to Wine-20080114 advapi32 - reactos/dll/win32/advapi32/crypt/*.c # ekohl says we're not sharing this

1 0

[winesync] 31799: Autosyncing with Wine HEAD
by winesync＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: winesync Date: Mon Jan 14 20:07:49 2008 New Revision: 31799 URL: http://svn.reactos.org/svn/reactos?rev=31799&view=rev Log: Autosyncing with Wine HEAD Added: trunk/rostests/winetests/hlink/ trunk/rostests/winetests/hlink/hlink.c (with props) trunk/rostests/winetests/hlink/hlink.rbuild (with props) trunk/rostests/winetests/hlink/testlist.c (with props) trunk/rostests/winetests/schannel/ trunk/rostests/winetests/schannel/main.c (with props) trunk/rostests/winetests/schannel/schannel.rbuild (with props) trunk/rostests/winetests/schannel/testlist.c (with props) Modified: trunk/rostests/winetests/comctl32/comboex.c trunk/rostests/winetests/comctl32/comctl32.rbuild trunk/rostests/winetests/comctl32/comctl32_ros.diff trunk/rostests/winetests/comctl32/datetime.c trunk/rostests/winetests/comctl32/header.c trunk/rostests/winetests/comctl32/imagelist.c trunk/rostests/winetests/comctl32/listview.c trunk/rostests/winetests/comctl32/monthcal.c trunk/rostests/winetests/comctl32/mru.c trunk/rostests/winetests/comctl32/progress.c trunk/rostests/winetests/comctl32/rebar.c trunk/rostests/winetests/comctl32/treeview.c trunk/rostests/winetests/rsaenh/rsaenh.c trunk/rostests/winetests/rsaenh/rsaenh.rbuild trunk/rostests/winetests/shlwapi/shlwapi.rbuild trunk/rostests/winetests/shlwapi/string.c trunk/rostests/winetests/shlwapi/url.c trunk/rostests/winetests/urlmon/misc.c trunk/rostests/winetests/urlmon/protocol.c trunk/rostests/winetests/urlmon/url.c trunk/rostests/winetests/urlmon/urlmon.rbuild trunk/rostests/winetests/wininet/ftp.c trunk/rostests/winetests/wininet/generated.c trunk/rostests/winetests/wininet/url.c trunk/rostests/winetests/wininet/wininet.rbuild [This mail would be too long, it was shortened to contain the URLs only.] Modified: trunk/rostests/winetests/comctl32/comboex.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/comboe… Modified: trunk/rostests/winetests/comctl32/comctl32.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/comctl… Modified: trunk/rostests/winetests/comctl32/comctl32_ros.diff URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/comctl… Modified: trunk/rostests/winetests/comctl32/datetime.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/dateti… Modified: trunk/rostests/winetests/comctl32/header.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/header… Modified: trunk/rostests/winetests/comctl32/imagelist.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/imagel… Modified: trunk/rostests/winetests/comctl32/listview.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/listvi… Modified: trunk/rostests/winetests/comctl32/monthcal.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/monthc… Modified: trunk/rostests/winetests/comctl32/mru.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/mru.c?… Modified: trunk/rostests/winetests/comctl32/progress.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/progre… Modified: trunk/rostests/winetests/comctl32/rebar.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/rebar.… Modified: trunk/rostests/winetests/comctl32/treeview.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/treevi… Added: trunk/rostests/winetests/hlink/hlink.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/hlink/hlink.c?r… Added: trunk/rostests/winetests/hlink/hlink.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/hlink/hlink.rbu… Added: trunk/rostests/winetests/hlink/testlist.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/hlink/testlist.… Modified: trunk/rostests/winetests/rsaenh/rsaenh.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/rsaenh/rsaenh.c… Modified: trunk/rostests/winetests/rsaenh/rsaenh.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/rsaenh/rsaenh.r… Added: trunk/rostests/winetests/schannel/main.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/schannel/main.c… Added: trunk/rostests/winetests/schannel/schannel.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/schannel/schann… Added: trunk/rostests/winetests/schannel/testlist.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/schannel/testli… Modified: trunk/rostests/winetests/shlwapi/shlwapi.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/shlwapi/shlwapi… Modified: trunk/rostests/winetests/shlwapi/string.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/shlwapi/string.… Modified: trunk/rostests/winetests/shlwapi/url.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/shlwapi/url.c?r… Modified: trunk/rostests/winetests/urlmon/misc.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/urlmon/misc.c?r… Modified: trunk/rostests/winetests/urlmon/protocol.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/urlmon/protocol… Modified: trunk/rostests/winetests/urlmon/url.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/urlmon/url.c?re… Modified: trunk/rostests/winetests/urlmon/urlmon.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/urlmon/urlmon.r… Modified: trunk/rostests/winetests/wininet/ftp.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/wininet/ftp.c?r… Modified: trunk/rostests/winetests/wininet/generated.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/wininet/generat… Modified: trunk/rostests/winetests/wininet/url.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/wininet/url.c?r… Modified: trunk/rostests/winetests/wininet/wininet.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/wininet/wininet…

1 0

[winesync] 31798: Autosyncing with Wine HEAD
by winesync＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: winesync Date: Mon Jan 14 19:52:12 2008 New Revision: 31798 URL: http://svn.reactos.org/svn/reactos?rev=31798&view=rev Log: Autosyncing with Wine HEAD Modified: trunk/reactos/dll/win32/wldap32/ber.c trunk/reactos/dll/win32/wldap32/control.c trunk/reactos/dll/win32/wldap32/parse.c trunk/reactos/dll/win32/wldap32/wldap32.rbuild Modified: trunk/reactos/dll/win32/wldap32/ber.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wldap32/ber.c?re… ============================================================================== --- trunk/reactos/dll/win32/wldap32/ber.c (original) +++ trunk/reactos/dll/win32/wldap32/ber.c Mon Jan 14 19:52:12 2008 @@ -161,7 +161,7 @@ * Flatten a berelement structure into a berval structure. * * PARAMS - * berlement [I] Pointer to a berelement structure. + * berelement [I] Pointer to a berelement structure. * berval [O] Pointer to a berval structure. * * RETURNS @@ -187,7 +187,7 @@ * Free a berelement structure. * * PARAMS - * berlement [I] Pointer to the berelement structure to be freed. + * berelement [I] Pointer to the berelement structure to be freed. * buf [I] Flag. * * RETURNS @@ -245,7 +245,7 @@ * Failure: LBER_DEFAULT (no more data). * * NOTES - * len and cookie are intitialised by ber_first_element and should + * len and cookie are initialized by ber_first_element and should * be passed on in subsequent calls to ber_next_element. */ ULONG CDECL WLDAP32_ber_next_element( BerElement *berelement, ULONG *len, CHAR *opaque ) Modified: trunk/reactos/dll/win32/wldap32/control.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wldap32/control.… ============================================================================== --- trunk/reactos/dll/win32/wldap32/control.c (original) +++ trunk/reactos/dll/win32/wldap32/control.c Mon Jan 14 19:52:12 2008 @@ -247,7 +247,7 @@ * Failure: An LDAP error code. * * NOTES - * Pass the created control in conjuction with a sort control as + * Pass the created control in conjunction with a sort control as * server controls in subsequent calls to ldap_search_ext(_s). The * server will then return a sorted, contiguous subset of results * that meets the criteria specified in the LDAPVLVInfo structure. Modified: trunk/reactos/dll/win32/wldap32/parse.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wldap32/parse.c?… ============================================================================== --- trunk/reactos/dll/win32/wldap32/parse.c (original) +++ trunk/reactos/dll/win32/wldap32/parse.c Mon Jan 14 19:52:12 2008 @@ -409,7 +409,7 @@ * PARAMS * ld [I] Pointer to an LDAP context. * control [I] Controls obtained from a result message. - * targetpos [O] Positition of the target in the result list. + * targetpos [O] Position of the target in the result list. * listcount [O] Estimate of the number of results in the list. * context [O] Server side context. * errcode [O] Error code from the listview operation. Modified: trunk/reactos/dll/win32/wldap32/wldap32.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wldap32/wldap32.… ============================================================================== --- trunk/reactos/dll/win32/wldap32/wldap32.rbuild (original) +++ trunk/reactos/dll/win32/wldap32/wldap32.rbuild Mon Jan 14 19:52:12 2008 @@ -1,5 +1,6 @@ <?xml version="1.0"?> <!DOCTYPE module SYSTEM "../../../tools/rbuild/project.dtd"> +<group> <module name="wldap32" type="win32dll" baseaddress="${BASEADDRESS_WLDAP32}" installbase="system32" installname="wldap32.dll" allowwarnings="true"> <importlibrary definition="wldap32.spec.def" /> <include base="wldap32">.</include> @@ -34,3 +35,4 @@ <file>wldap32.rc</file> <file>wldap32.spec</file> </module> +</group>

1 0

[winesync] 31797: Autosyncing with Wine HEAD
by winesync＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: winesync Date: Mon Jan 14 19:50:34 2008 New Revision: 31797 URL: http://svn.reactos.org/svn/reactos?rev=31797&view=rev Log: Autosyncing with Wine HEAD Modified: trunk/reactos/dll/win32/wininet/ftp.c trunk/reactos/dll/win32/wininet/http.c trunk/reactos/dll/win32/wininet/internet.c trunk/reactos/dll/win32/wininet/urlcache.c trunk/reactos/dll/win32/wininet/wininet.rbuild trunk/reactos/dll/win32/wininet/wininet_ros.diff Modified: trunk/reactos/dll/win32/wininet/ftp.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wininet/ftp.c?re… ============================================================================== --- trunk/reactos/dll/win32/wininet/ftp.c (original) +++ trunk/reactos/dll/win32/wininet/ftp.c Mon Jan 14 19:50:34 2008 @@ -845,13 +845,12 @@ } } ret = FtpGetCurrentDirectoryW(hFtpSession, lpszCurrentDirectory?dir:NULL, lpdwCurrentDirectory?&len:NULL); - if(lpdwCurrentDirectory) { - *lpdwCurrentDirectory = len; - if(lpszCurrentDirectory) { - WideCharToMultiByte(CP_ACP, 0, dir, len, lpszCurrentDirectory, *lpdwCurrentDirectory, NULL, NULL); - HeapFree(GetProcessHeap(), 0, dir); - } - } + + if (ret && lpszCurrentDirectory) + WideCharToMultiByte(CP_ACP, 0, dir, -1, lpszCurrentDirectory, len, NULL, NULL); + + if (lpdwCurrentDirectory) *lpdwCurrentDirectory = len; + HeapFree(GetProcessHeap(), 0, dir); return ret; } @@ -886,9 +885,27 @@ TRACE("len(%d)\n", *lpdwCurrentDirectory); lpwfs = (LPWININETFTPSESSIONW) WININET_GetObject( hFtpSession ); - if (NULL == lpwfs || WH_HFTPSESSION != lpwfs->hdr.htype) + if (NULL == lpwfs) + { + INTERNET_SetLastError(ERROR_INVALID_HANDLE); + goto lend; + } + + if (WH_HFTPSESSION != lpwfs->hdr.htype) { INTERNET_SetLastError(ERROR_INTERNET_INCORRECT_HANDLE_TYPE); + goto lend; + } + + if (!lpdwCurrentDirectory) + { + INTERNET_SetLastError(ERROR_INVALID_PARAMETER); + goto lend; + } + + if (lpszCurrentDirectory == NULL) + { + INTERNET_SetLastError(ERROR_INSUFFICIENT_BUFFER); goto lend; } @@ -948,8 +965,6 @@ /* Clear any error information */ INTERNET_SetLastError(0); - ZeroMemory(lpszCurrentDirectory, *lpdwCurrentDirectory); - hIC = lpwfs->lpAppInfo; if (!FTP_SendCommand(lpwfs->sndSocket, FTP_CMD_PWD, NULL, lpwfs->hdr.lpfnStatusCB, &lpwfs->hdr, lpwfs->hdr.dwContext)) @@ -961,7 +976,7 @@ if (nResCode == 257) /* Extract directory name */ { DWORD firstpos, lastpos, len; - LPWSTR lpszResponseBuffer = WININET_strdup_AtoW(INTERNET_GetResponseBuffer()); + LPWSTR lpszResponseBuffer = WININET_strdup_AtoW(INTERNET_GetResponseBuffer()); for (firstpos = 0, lastpos = 0; lpszResponseBuffer[lastpos]; lastpos++) { @@ -971,14 +986,19 @@ firstpos = lastpos; else break; - } + } } - - len = lastpos - firstpos - 1; - lstrcpynW(lpszCurrentDirectory, &lpszResponseBuffer[firstpos+1], *lpdwCurrentDirectory); + len = lastpos - firstpos; + if (*lpdwCurrentDirectory >= len) + { + memcpy(lpszCurrentDirectory, &lpszResponseBuffer[firstpos + 1], len * sizeof(WCHAR)); + lpszCurrentDirectory[len - 1] = 0; + *lpdwCurrentDirectory = len; + bSuccess = TRUE; + } + else INTERNET_SetLastError(ERROR_INSUFFICIENT_BUFFER); + HeapFree(GetProcessHeap(), 0, lpszResponseBuffer); - *lpdwCurrentDirectory = len; - bSuccess = TRUE; } else FTP_SetResponseError(nResCode); @@ -2335,7 +2355,7 @@ if(lpszResponse[3] != '-') break; else - { /* Start of multiline repsonse. Loop until we get "nnn " */ + { /* Start of multiline response. Loop until we get "nnn " */ multiline = TRUE; memcpy(firstprefix, lpszResponse, 3); firstprefix[3] = ' '; @@ -3411,7 +3431,7 @@ TRACE("\n"); - /* Allocate intial file properties array */ + /* Allocate initial file properties array */ *lpafp = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(FILEPROPERTIESW)*(sizeFilePropArray)); if (!*lpafp) return FALSE; @@ -3443,7 +3463,7 @@ tmpafp = HeapReAlloc(GetProcessHeap(), 0, *lpafp, sizeof(FILEPROPERTIESW)*indexFilePropArray); - if (NULL == tmpafp) + if (NULL != tmpafp) *lpafp = tmpafp; } *dwfp = indexFilePropArray; Modified: trunk/reactos/dll/win32/wininet/http.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wininet/http.c?r… ============================================================================== --- trunk/reactos/dll/win32/wininet/http.c (original) +++ trunk/reactos/dll/win32/wininet/http.c Mon Jan 14 19:50:34 2008 @@ -264,7 +264,7 @@ LPWSTR requestString; DWORD len, n; LPCWSTR *req; - INT i; + UINT i; LPWSTR p; static const WCHAR szSpace[] = { ' ',0 }; @@ -283,7 +283,7 @@ req[n++] = path; req[n++] = http1_1 ? g_szHttp1_1 : g_szHttp1_0; - /* Append custom request heades */ + /* Append custom request headers */ for (i = 0; i < lpwhr->nCustHeaders; i++) { if (lpwhr->pCustHeaders[i].wFlags & HDR_ISREQUEST) @@ -725,7 +725,7 @@ } /* read any content returned by the server so that the connection can be - * resued */ + * reused */ static void HTTP_DrainContent(LPWININETHTTPREQW lpwhr) { DWORD bytes_read; @@ -1415,7 +1415,7 @@ } if (NULL != lpszReferrer && strlenW(lpszReferrer)) - HTTP_ProcessHeader(lpwhr, HTTP_REFERER, lpszReferrer, HTTP_ADDHDR_FLAG_COALESCE); + HTTP_ProcessHeader(lpwhr, HTTP_REFERER, lpszReferrer, HTTP_ADDREQ_FLAG_ADD | HTTP_ADDHDR_FLAG_REQ); if (lpszAcceptTypes) { @@ -1438,22 +1438,7 @@ else if (strlenW(lpszVerb)) lpwhr->lpszVerb = WININET_strdupW(lpszVerb); - if (NULL != lpszReferrer && strlenW(lpszReferrer)) - { - WCHAR buf[MAXHOSTNAME]; - URL_COMPONENTSW UrlComponents; - - memset( &UrlComponents, 0, sizeof UrlComponents ); - UrlComponents.dwStructSize = sizeof UrlComponents; - UrlComponents.lpszHostName = buf; - UrlComponents.dwHostNameLength = MAXHOSTNAME; - - InternetCrackUrlW(lpszReferrer, 0, 0, &UrlComponents); - if (strlenW(UrlComponents.lpszHostName)) - HTTP_ProcessHeader(lpwhr, szHost, UrlComponents.lpszHostName, HTTP_ADDREQ_FLAG_ADD | HTTP_ADDREQ_FLAG_REPLACE | HTTP_ADDHDR_FLAG_REQ); - } - else - HTTP_ProcessHeader(lpwhr, szHost, lpwhs->lpszHostName, HTTP_ADDREQ_FLAG_ADD | HTTP_ADDREQ_FLAG_REPLACE | HTTP_ADDHDR_FLAG_REQ); + HTTP_ProcessHeader(lpwhr, szHost, lpwhs->lpszHostName, HTTP_ADDREQ_FLAG_ADD | HTTP_ADDHDR_FLAG_REQ); if (lpwhs->nServerPort == INTERNET_INVALID_PORT_NUMBER) lpwhs->nServerPort = (dwFlags & INTERNET_FLAG_SECURE ? @@ -1785,7 +1770,7 @@ if (index >= 0) lphttpHdr = &lpwhr->pCustHeaders[index]; - /* Ensure header satisifies requested attributes */ + /* Ensure header satisfies requested attributes */ if (!lphttpHdr || ((dwInfoLevel & HTTP_QUERY_FLAG_REQUEST_HEADERS) && (~lphttpHdr->wFlags & HDR_ISREQUEST))) @@ -1797,7 +1782,7 @@ if (lpdwIndex) (*lpdwIndex)++; - /* coalesce value to reuqested type */ + /* coalesce value to requested type */ if (dwInfoLevel & HTTP_QUERY_FLAG_NUMBER) { *(int *)lpBuffer = atoiW(lphttpHdr->lpszValue); @@ -3032,6 +3017,7 @@ a bit different */ NETCON_recv(&lpwhr->netConnection, buffer, buflen, MSG_PEEK, &rc); #endif + /* * We should first receive 'HTTP/1.x nnn OK' where nnn is the status code. */ Modified: trunk/reactos/dll/win32/wininet/internet.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wininet/internet… ============================================================================== --- trunk/reactos/dll/win32/wininet/internet.c (original) +++ trunk/reactos/dll/win32/wininet/internet.c Mon Jan 14 19:50:34 2008 @@ -1070,7 +1070,8 @@ TRACE("(%s %u %x %p)\n", debugstr_a(lpszUrl), dwUrlLength, dwFlags, lpUrlComponents); - if (!lpszUrl || !*lpszUrl) + if (!lpszUrl || !*lpszUrl || !lpUrlComponents || + lpUrlComponents->dwStructSize != sizeof(URL_COMPONENTSA)) { INTERNET_SetLastError(ERROR_INVALID_PARAMETER); return FALSE; @@ -1088,6 +1089,7 @@ MultiByteToWideChar(CP_ACP,0,lpszUrl,dwUrlLength,lpwszUrl,nLength); memset(&UCW,0,sizeof(UCW)); + UCW.dwStructSize = sizeof(URL_COMPONENTSW); if(lpUrlComponents->dwHostNameLength!=0) UCW.dwHostNameLength= lpUrlComponents->dwHostNameLength; if(lpUrlComponents->dwUserNameLength!=0) @@ -2329,6 +2331,20 @@ } } break; + case INTERNET_OPTION_VERSION: + { + TRACE("INTERNET_OPTION_VERSION\n"); + if (*lpdwBufferLength < sizeof(INTERNET_VERSION_INFO)) + INTERNET_SetLastError(ERROR_INSUFFICIENT_BUFFER); + else + { + static const INTERNET_VERSION_INFO info = { 6, 0 }; + memcpy(lpBuffer, &info, sizeof(info)); + *lpdwBufferLength = sizeof(info); + bSuccess = TRUE; + } + break; + } default: FIXME("Stub! %d\n", dwOption); break; @@ -2793,7 +2809,7 @@ if (lpszUrl == NULL) { /* - * According to the doc we are supost to use the ip for the next + * According to the doc we are supposed to use the ip for the next * server in the WnInet internal server database. I have * no idea what that is or how to get it. * Modified: trunk/reactos/dll/win32/wininet/urlcache.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wininet/urlcache… ============================================================================== --- trunk/reactos/dll/win32/wininet/urlcache.c (original) +++ trunk/reactos/dll/win32/wininet/urlcache.c Mon Jan 14 19:50:34 2008 @@ -113,7 +113,7 @@ /* packing to dword align start of next field */ /* CHAR szSourceUrlName[]; (url) */ /* packing to dword align start of next field */ - /* CHAR szLocalFileName[]; (local file name exluding path) */ + /* CHAR szLocalFileName[]; (local file name excluding path) */ /* packing to dword align start of next field */ /* CHAR szHeaderInfo[]; (header info) */ } URL_CACHEFILE_ENTRY; @@ -171,7 +171,7 @@ LPWSTR path; /* path to url container directory */ HANDLE hMapping; /* handle of file mapping */ DWORD file_size; /* size of file when mapping was opened */ - HANDLE hMutex; /* hande of mutex */ + HANDLE hMutex; /* handle of mutex */ } URLCACHECONTAINER; @@ -2796,6 +2796,7 @@ LPINTERNET_CACHE_ENTRY_INFOW lpFirstCacheEntryInfo, LPDWORD lpdwFirstCacheEntryInfoBufferSize) { FIXME("(%s, %p, %p): stub\n", debugstr_w(lpszUrlSearchPattern), lpFirstCacheEntryInfo, lpdwFirstCacheEntryInfoBufferSize); + SetLastError(ERROR_FILE_NOT_FOUND); return 0; } Modified: trunk/reactos/dll/win32/wininet/wininet.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wininet/wininet.… ============================================================================== --- trunk/reactos/dll/win32/wininet/wininet.rbuild (original) +++ trunk/reactos/dll/win32/wininet/wininet.rbuild Mon Jan 14 19:50:34 2008 @@ -1,5 +1,6 @@ <?xml version="1.0"?> <!DOCTYPE module SYSTEM "../../../tools/rbuild/project.dtd"> +<group> <module name="wininet" type="win32dll" baseaddress="${BASEADDRESS_WININET}" installbase="system32" installname="wininet.dll" allowwarnings="true"> <autoregister infsection="OleControlDlls" type="DllInstall" /> <importlibrary definition="wininet.spec.def" /> @@ -33,3 +34,4 @@ <file>version.rc</file> <file>wininet.spec</file> </module> +</group> Modified: trunk/reactos/dll/win32/wininet/wininet_ros.diff URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wininet/wininet_… ============================================================================== --- trunk/reactos/dll/win32/wininet/wininet_ros.diff (original) +++ trunk/reactos/dll/win32/wininet/wininet_ros.diff Mon Jan 14 19:50:34 2008 @@ -23,7 +23,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(wininet); static const WCHAR g_szHttp1_0[] = {' ','H','T','T','P','/','1','.','0',0 }; -@@ -3027,14 +3028,21 @@ +@@ -3010,7 +3011,11 @@ /* * HACK peek at the buffer */ @@ -32,10 +32,10 @@ + a bit different */ NETCON_recv(&lpwhr->netConnection, buffer, buflen, MSG_PEEK, &rc); +#endif + /* * We should first receive 'HTTP/1.x nnn OK' where nnn is the status code. - */ - buflen = MAX_REPLY_LEN; +@@ -3019,6 +3024,9 @@ memset(buffer, 0, MAX_REPLY_LEN); if (!NETCON_getNextLine(&lpwhr->netConnection, bufferA, &buflen)) goto lend; @@ -305,7 +305,7 @@ =================================================================== --- wininet.rbuild (revision 30893) +++ wininet.rbuild (working copy) -@@ -18,6 +18,7 @@ +@@ -19,6 +19,7 @@ <library>ntdll</library> <library>secur32</library> <library>crypt32</library>

1 0

[winesync] 31796: Autosyncing with Wine HEAD
by winesync＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: winesync Date: Mon Jan 14 19:39:00 2008 New Revision: 31796 URL: http://svn.reactos.org/svn/reactos?rev=31796&view=rev Log: Autosyncing with Wine HEAD Modified: trunk/reactos/dll/win32/urlmon/bindctx.c trunk/reactos/dll/win32/urlmon/binding.c trunk/reactos/dll/win32/urlmon/bindprot.c trunk/reactos/dll/win32/urlmon/file.c trunk/reactos/dll/win32/urlmon/format.c trunk/reactos/dll/win32/urlmon/ftp.c trunk/reactos/dll/win32/urlmon/http.c trunk/reactos/dll/win32/urlmon/internet.c trunk/reactos/dll/win32/urlmon/mk.c trunk/reactos/dll/win32/urlmon/regsvr.c trunk/reactos/dll/win32/urlmon/sec_mgr.c trunk/reactos/dll/win32/urlmon/session.c trunk/reactos/dll/win32/urlmon/umon.c trunk/reactos/dll/win32/urlmon/umstream.c trunk/reactos/dll/win32/urlmon/urlmon.rbuild trunk/reactos/dll/win32/urlmon/urlmon_main.c trunk/reactos/dll/win32/urlmon/urlmon_main.h Modified: trunk/reactos/dll/win32/urlmon/bindctx.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/bindctx.c… ============================================================================== --- trunk/reactos/dll/win32/urlmon/bindctx.c (original) +++ trunk/reactos/dll/win32/urlmon/bindctx.c Mon Jan 14 19:39:00 2008 @@ -16,23 +16,10 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> #include <stdio.h> -#define COBJMACROS -#define NONAMELESSUNION -#define NONAMELESSSTRUCT - -#include "windef.h" -#include "winbase.h" -#include "objbase.h" -#include "winuser.h" -#include "ole2.h" -#include "urlmon.h" #include "urlmon_main.h" - #include "wine/debug.h" -#include "wine/unicode.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -132,7 +119,7 @@ if(This->authenticate) IAuthenticate_Release(This->authenticate); IBindStatusCallback_Release(This->callback); - urlmon_free(This); + heap_free(This); } return ref; @@ -455,7 +442,7 @@ static IBindStatusCallback *create_bsc(IBindStatusCallback *bsc) { - BindStatusCallback *ret = urlmon_alloc_zero(sizeof(BindStatusCallback)); + BindStatusCallback *ret = heap_alloc_zero(sizeof(BindStatusCallback)); ret->lpBindStatusCallbackVtbl = &BindStatusCallbackVtbl; ret->lpServiceProviderVtbl = &BSCServiceProviderVtbl; @@ -640,7 +627,7 @@ if(!ref) { IBindCtx_Release(This->bindctx); - urlmon_free(This); + heap_free(This); } return ref; @@ -718,13 +705,13 @@ return IBindCtx_GetObjectParam(This->bindctx, pszkey, punk); } -static HRESULT WINAPI AsyncBindCtx_RevokeObjectParam(IBindCtx *iface, LPOLESTR ppenum) -{ - AsyncBindCtx *This = BINDCTX_THIS(iface); - - TRACE("(%p)->(%p)\n", This, ppenum); - - return IBindCtx_RevokeObjectParam(This->bindctx, ppenum); +static HRESULT WINAPI AsyncBindCtx_RevokeObjectParam(IBindCtx *iface, LPOLESTR pszkey) +{ + AsyncBindCtx *This = BINDCTX_THIS(iface); + + TRACE("(%p)->(%s)\n", This, debugstr_w(pszkey)); + + return IBindCtx_RevokeObjectParam(This->bindctx, pszkey); } static HRESULT WINAPI AsyncBindCtx_EnumObjectParam(IBindCtx *iface, IEnumString **pszkey) @@ -833,11 +820,16 @@ if(reserved) WARN("reserved=%d\n", reserved); - hres = CreateBindCtx(0, &bindctx); - if(FAILED(hres)) - return hres; - - ret = urlmon_alloc(sizeof(AsyncBindCtx)); + if(ibind) { + IBindCtx_AddRef(ibind); + bindctx = ibind; + }else { + hres = CreateBindCtx(0, &bindctx); + if(FAILED(hres)) + return hres; + } + + ret = heap_alloc(sizeof(AsyncBindCtx)); ret->lpBindCtxVtbl = &AsyncBindCtxVtbl; ret->ref = 1; Modified: trunk/reactos/dll/win32/urlmon/binding.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/binding.c… ============================================================================== --- trunk/reactos/dll/win32/urlmon/binding.c (original) +++ trunk/reactos/dll/win32/urlmon/binding.c Mon Jan 14 19:39:00 2008 @@ -16,23 +16,15 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> - -#define COBJMACROS -#define NONAMELESSUNION -#define NONAMELESSSTRUCT - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" -#include "ole2.h" -#include "urlmon.h" #include "urlmon_main.h" +#include "winreg.h" #include "wine/debug.h" -#include "wine/unicode.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); + +static WCHAR cbinding_contextW[] = {'C','B','i','n','d','i','n','g',' ','C','o','n','t','e','x','t',0}; +static WCHAR bscb_holderW[] = { '_','B','S','C','B','_','H','o','l','d','e','r','_',0 }; typedef struct Binding Binding; @@ -64,6 +56,10 @@ END_DOWNLOAD } download_state_t; +#define BINDING_LOCKED 0x0001 +#define BINDING_STOPPED 0x0002 +#define BINDING_OBJAVAIL 0x0004 + struct Binding { const IBindingVtbl *lpBindingVtbl; const IInternetProtocolSinkVtbl *lpInternetProtocolSinkVtbl; @@ -79,12 +75,19 @@ BINDINFO bindinfo; DWORD bindf; + BOOL to_object; LPWSTR mime; + UINT clipboard_format; LPWSTR url; + IID iid; BOOL report_mime; DWORD continue_call; - BOOL request_locked; + DWORD state; + HRESULT hres; download_state_t download_state; + IUnknown *obj; + IMoniker *mon; + IBindCtx *bctx; DWORD apartment_thread; HWND notif_hwnd; @@ -251,6 +254,212 @@ ); } +static void set_binding_mime(Binding *binding, LPCWSTR mime) +{ + EnterCriticalSection(&binding->section); + + if(binding->report_mime) { + heap_free(binding->mime); + binding->mime = heap_strdupW(mime); + } + + LeaveCriticalSection(&binding->section); +} + +static void handle_mime_available(Binding *binding, BOOL verify) +{ + BOOL report_mime; + + EnterCriticalSection(&binding->section); + report_mime = binding->report_mime; + binding->report_mime = FALSE; + LeaveCriticalSection(&binding->section); + + if(!report_mime) + return; + + if(verify) { + LPWSTR mime = NULL; + + fill_stream_buffer(binding->stream); + FindMimeFromData(NULL, binding->url, binding->stream->buf, + min(binding->stream->buf_size, 255), binding->mime, 0, &mime, 0); + + heap_free(binding->mime); + binding->mime = heap_strdupW(mime); + CoTaskMemFree(mime); + } + + IBindStatusCallback_OnProgress(binding->callback, 0, 0, BINDSTATUS_MIMETYPEAVAILABLE, binding->mime); + + binding->clipboard_format = RegisterClipboardFormatW(binding->mime); +} + +typedef struct { + task_header_t header; + BOOL verify; +} mime_available_task_t; + +static void mime_available_proc(Binding *binding, task_header_t *t) +{ + mime_available_task_t *task = (mime_available_task_t*)t; + + handle_mime_available(binding, task->verify); + + heap_free(task); +} + +static void mime_available(Binding *This, LPCWSTR mime, BOOL verify) +{ + if(mime) + set_binding_mime(This, mime); + + if(GetCurrentThreadId() == This->apartment_thread) { + handle_mime_available(This, verify); + }else { + mime_available_task_t *task = heap_alloc(sizeof(task_header_t)); + task->verify = verify; + push_task(This, &task->header, mime_available_proc); + } +} + +static void stop_binding(Binding *binding, HRESULT hres, LPCWSTR str) +{ + if(binding->state & BINDING_LOCKED) { + IInternetProtocol_UnlockRequest(binding->protocol); + binding->state &= ~BINDING_LOCKED; + } + + if(!(binding->state & BINDING_STOPPED)) { + binding->state |= BINDING_STOPPED; + + IBindStatusCallback_OnStopBinding(binding->callback, hres, str); + binding->hres = hres; + } +} + +static LPWSTR get_mime_clsid(LPCWSTR mime, CLSID *clsid) +{ + LPWSTR key_name, ret; + DWORD res, type, size; + HKEY hkey; + int len; + HRESULT hres; + + static const WCHAR mime_keyW[] = + {'M','I','M','E','\\','D','a','t','a','b','a','s','e','\\', + 'C','o','n','t','e','n','t',' ','T','y','p','e','\\'}; + static const WCHAR clsidW[] = {'C','L','S','I','D',0}; + + len = strlenW(mime)+1; + key_name = heap_alloc(sizeof(mime_keyW) + len*sizeof(WCHAR)); + memcpy(key_name, mime_keyW, sizeof(mime_keyW)); + strcpyW(key_name + sizeof(mime_keyW)/sizeof(WCHAR), mime); + + res = RegOpenKeyW(HKEY_CLASSES_ROOT, key_name, &hkey); + heap_free(key_name); + if(res != ERROR_SUCCESS) { + WARN("Could not open MIME key: %x\n", res); + return NULL; + } + + size = 50*sizeof(WCHAR); + ret = heap_alloc(size); + res = RegQueryValueExW(hkey, clsidW, NULL, &type, (LPBYTE)ret, &size); + RegCloseKey(hkey); + if(res != ERROR_SUCCESS) { + WARN("Could not get CLSID: %08x\n", res); + heap_free(ret); + return NULL; + } + + hres = CLSIDFromString(ret, clsid); + if(FAILED(hres)) { + WARN("Could not parse CLSID: %08x\n", hres); + heap_free(ret); + return NULL; + } + + return ret; +} + +static void load_doc_mon(Binding *binding, IPersistMoniker *persist) +{ + IBindCtx *bctx; + HRESULT hres; + + hres = CreateAsyncBindCtxEx(binding->bctx, 0, NULL, NULL, &bctx, 0); + if(FAILED(hres)) { + WARN("CreateAsyncBindCtxEx failed: %08x\n", hres); + return; + } + + IBindCtx_RevokeObjectParam(bctx, bscb_holderW); + IBindCtx_RegisterObjectParam(bctx, cbinding_contextW, (IUnknown*)BINDING(binding)); + + hres = IPersistMoniker_Load(persist, binding->download_state == END_DOWNLOAD, binding->mon, bctx, 0x12); + IBindCtx_RevokeObjectParam(bctx, cbinding_contextW); + IBindCtx_Release(bctx); + if(FAILED(hres)) + FIXME("Load failed: %08x\n", hres); +} + +static void create_object(Binding *binding) +{ + IPersistMoniker *persist; + LPWSTR clsid_str; + CLSID clsid; + HRESULT hres; + + if(!binding->mime) { + FIXME("MIME unavailable\n"); + return; + } + + if(!(clsid_str = get_mime_clsid(binding->mime, &clsid))) { + FIXME("Could not find object for MIME %s\n", debugstr_w(binding->mime)); + return; + } + + IBindStatusCallback_OnProgress(binding->callback, 0, 0, BINDSTATUS_CLASSIDAVAILABLE, clsid_str); + + IBindStatusCallback_OnProgress(binding->callback, 0, 0, BINDSTATUS_BEGINSYNCOPERATION, NULL); + + hres = CoCreateInstance(&clsid, NULL, CLSCTX_INPROC_SERVER|CLSCTX_INPROC_HANDLER, + &binding->iid, (void**)&binding->obj); + if(FAILED(hres)) + FIXME("CoCreateInstance failed: %08x\n", hres); + + binding->state |= BINDING_OBJAVAIL; + + hres = IUnknown_QueryInterface(binding->obj, &IID_IPersistMoniker, (void**)&persist); + if(SUCCEEDED(hres)) { + IMonikerProp *prop; + + hres = IPersistMoniker_QueryInterface(persist, &IID_IMonikerProp, (void**)&prop); + if(SUCCEEDED(hres)) { + IMonikerProp_PutProperty(prop, MIMETYPEPROP, binding->mime); + IMonikerProp_PutProperty(prop, CLASSIDPROP, clsid_str); + IMonikerProp_Release(prop); + } + + load_doc_mon(binding, persist); + + IPersistMoniker_Release(persist); + }else { + FIXME("Could not get IPersistMoniker: %08x\n", hres); + /* FIXME: Try query IPersistFile */ + } + + heap_free(clsid_str); + + IBindStatusCallback_OnObjectAvailable(binding->callback, &binding->iid, binding->obj); + + IBindStatusCallback_OnProgress(binding->callback, 0, 0, BINDSTATUS_ENDSYNCOPERATION, NULL); + + stop_binding(binding, S_OK, NULL); +} + #define STREAM_THIS(iface) DEFINE_THIS(ProtocolStream, Stream, iface) static HRESULT WINAPI ProtocolStream_QueryInterface(IStream *iface, @@ -299,7 +508,7 @@ if(!ref) { IInternetProtocol_Release(This->protocol); - urlmon_free(This); + heap_free(This); URLMON_UnlockModule(); } @@ -451,7 +660,7 @@ static ProtocolStream *create_stream(IInternetProtocol *protocol) { - ProtocolStream *ret = urlmon_alloc(sizeof(ProtocolStream)); + ProtocolStream *ret = heap_alloc(sizeof(ProtocolStream)); ret->lpStreamVtbl = &ProtocolStreamVtbl; ret->ref = 1; @@ -519,6 +728,8 @@ if(!ref) { if (This->notif_hwnd) DestroyWindow( This->notif_hwnd ); + if(This->mon) + IMoniker_Release(This->mon); if(This->callback) IBindStatusCallback_Release(This->callback); if(This->protocol) @@ -527,14 +738,18 @@ IServiceProvider_Release(This->service_provider); if(This->stream) IStream_Release(STREAM(This->stream)); + if(This->obj) + IUnknown_Release(This->obj); + if(This->bctx) + IBindCtx_Release(This->bctx); ReleaseBindInfo(&This->bindinfo); This->section.DebugInfo->Spare[0] = 0; DeleteCriticalSection(&This->section); - urlmon_free(This->mime); - urlmon_free(This->url); - - urlmon_free(This); + heap_free(This->mime); + heap_free(This->url); + + heap_free(This); URLMON_UnlockModule(); } @@ -583,6 +798,25 @@ Binding *This = BINDING_THIS(iface); FIXME("(%p)->(%p %p %p %p)\n", This, pclsidProtocol, pdwResult, pszResult, pdwReserved); return E_NOTIMPL; +} + +static Binding *get_bctx_binding(IBindCtx *bctx) +{ + IBinding *binding; + IUnknown *unk; + HRESULT hres; + + hres = IBindCtx_GetObjectParam(bctx, cbinding_contextW, &unk); + if(FAILED(hres)) + return NULL; + + hres = IUnknown_QueryInterface(unk, &IID_IBinding, (void*)&binding); + IUnknown_Release(unk); + if(FAILED(hres)) + return NULL; + + /* FIXME!!! */ + return BINDING_THIS(binding); } #undef BINDING_THIS @@ -631,7 +865,7 @@ IInternetProtocol_Continue(binding->protocol, &task->data); - urlmon_free(task); + heap_free(task); } static HRESULT WINAPI InternetProtocolSink_Switch(IInternetProtocolSink *iface, @@ -642,7 +876,7 @@ TRACE("(%p)->(%p)\n", This, pProtocolData); - task = urlmon_alloc(sizeof(switch_task_t)); + task = heap_alloc(sizeof(switch_task_t)); memcpy(&task->data, pProtocolData, sizeof(PROTOCOLDATA)); push_task(This, &task->header, switch_proc); @@ -670,8 +904,8 @@ IBindStatusCallback_OnProgress(binding->callback, task->progress, task->progress_max, task->status_code, task->status_text); - urlmon_free(task->status_text); - urlmon_free(task); + heap_free(task->status_text); + heap_free(task); } static void on_progress(Binding *This, ULONG progress, ULONG progress_max, @@ -685,7 +919,7 @@ return; } - task = urlmon_alloc(sizeof(on_progress_task_t)); + task = heap_alloc(sizeof(on_progress_task_t)); task->progress = progress; task->progress_max = progress_max; @@ -694,7 +928,7 @@ if(status_text) { DWORD size = (strlenW(status_text)+1)*sizeof(WCHAR); - task->status_text = urlmon_alloc(size); + task->status_text = heap_alloc(size); memcpy(task->status_text, status_text, size); }else { task->status_text = NULL; @@ -725,18 +959,16 @@ case BINDSTATUS_BEGINDOWNLOADDATA: fill_stream_buffer(This->stream); break; - case BINDSTATUS_MIMETYPEAVAILABLE: { - int len = strlenW(szStatusText)+1; - This->mime = urlmon_alloc(len*sizeof(WCHAR)); - memcpy(This->mime, szStatusText, len*sizeof(WCHAR)); + case BINDSTATUS_MIMETYPEAVAILABLE: + set_binding_mime(This, szStatusText); break; - } case BINDSTATUS_SENDINGREQUEST: on_progress(This, 0, 0, BINDSTATUS_SENDINGREQUEST, szStatusText); break; + case BINDSTATUS_PROTOCOLCLASSID: + break; case BINDSTATUS_VERIFIEDMIMETYPEAVAILABLE: - This->report_mime = FALSE; - on_progress(This, 0, 0, BINDSTATUS_MIMETYPEAVAILABLE, szStatusText); + mime_available(This, szStatusText, FALSE); break; case BINDSTATUS_CACHEFILENAMEAVAILABLE: break; @@ -762,21 +994,10 @@ return; if(GetCurrentThreadId() != This->apartment_thread) - FIXME("called from worked hread\n"); - - if(This->report_mime) { - LPWSTR mime; - - This->report_mime = FALSE; - - fill_stream_buffer(This->stream); - - FindMimeFromData(NULL, This->url, This->stream->buf, - min(This->stream->buf_size, 255), This->mime, 0, &mime, 0); - - IBindStatusCallback_OnProgress(This->callback, progress, progress_max, - BINDSTATUS_MIMETYPEAVAILABLE, mime); - } + FIXME("called from worker thread\n"); + + if(This->report_mime) + mime_available(This, NULL, TRUE); if(This->download_state == BEFORE_DOWNLOAD) { fill_stream_buffer(This->stream); @@ -796,16 +1017,22 @@ BINDSTATUS_DOWNLOADINGDATA, This->url); } - if(!This->request_locked) { - HRESULT hres = IInternetProtocol_LockRequest(This->protocol, 0); - This->request_locked = SUCCEEDED(hres); - } - - IBindStatusCallback_OnDataAvailable(This->callback, bscf, progress, - &formatetc, &This->stgmed); - - if(This->download_state == END_DOWNLOAD) { - IBindStatusCallback_OnStopBinding(This->callback, S_OK, NULL); + if(This->to_object) { + if(!(This->state & BINDING_OBJAVAIL)) + create_object(This); + }else { + if(!(This->state & BINDING_LOCKED)) { + HRESULT hres = IInternetProtocol_LockRequest(This->protocol, 0); + if(SUCCEEDED(hres)) + This->state |= BINDING_LOCKED; + } + + formatetc.cfFormat = This->clipboard_format; + IBindStatusCallback_OnDataAvailable(This->callback, bscf, progress, + &formatetc, &This->stgmed); + + if(This->download_state == END_DOWNLOAD) + stop_binding(This, S_OK, NULL); } } @@ -822,7 +1049,7 @@ report_data(binding, task->bscf, task->progress, task->progress_max); - urlmon_free(task); + heap_free(task); } static HRESULT WINAPI InternetProtocolSink_ReportData(IInternetProtocolSink *iface, @@ -836,7 +1063,7 @@ FIXME("called from worked hread\n"); if(This->continue_call) { - report_data_task_t *task = urlmon_alloc(sizeof(report_data_task_t)); + report_data_task_t *task = heap_alloc(sizeof(report_data_task_t)); task->bscf = grfBSCF; task->progress = ulProgress; task->progress_max = ulProgressMax; @@ -849,16 +1076,23 @@ return S_OK; } +typedef struct { + task_header_t header; + + HRESULT hres; + LPWSTR str; +} report_result_task_t; + static void report_result_proc(Binding *binding, task_header_t *t) { + report_result_task_t *task = (report_result_task_t*)t; + IInternetProtocol_Terminate(binding->protocol, 0); - if(binding->request_locked) { - IInternetProtocol_UnlockRequest(binding->protocol); - binding->request_locked = FALSE; - } - - urlmon_free(t); + stop_binding(binding, task->hres, task->str); + + heap_free(task->str); + heap_free(task); } static HRESULT WINAPI InternetProtocolSink_ReportResult(IInternetProtocolSink *iface, @@ -870,9 +1104,14 @@ if(GetCurrentThreadId() == This->apartment_thread && !This->continue_call) { IInternetProtocol_Terminate(This->protocol, 0); + stop_binding(This, hrResult, szResult); }else { - task_header_t *task = urlmon_alloc(sizeof(task_header_t)); - push_task(This, task, report_result_proc); + report_result_task_t *task = heap_alloc(sizeof(report_result_task_t)); + + task->hres = hrResult; + task->str = heap_strdupW(szResult); + + push_task(This, &task->header, report_result_proc); } return S_OK; @@ -1038,42 +1277,13 @@ IUnknown *unk; HRESULT hres; - static WCHAR wszBSCBHolder[] = { '_','B','S','C','B','_','H','o','l','d','e','r','_',0 }; - - hres = IBindCtx_GetObjectParam(pbc, wszBSCBHolder, &unk); + hres = IBindCtx_GetObjectParam(pbc, bscb_holderW, &unk); if(SUCCEEDED(hres)) { hres = IUnknown_QueryInterface(unk, &IID_IBindStatusCallback, (void**)callback); IUnknown_Release(unk); } return SUCCEEDED(hres) ? S_OK : MK_E_SYNTAX; -} - -static HRESULT get_protocol(Binding *This, LPCWSTR url) -{ - IClassFactory *cf = NULL; - HRESULT hres; - - hres = IBindStatusCallback_QueryInterface(This->callback, &IID_IInternetProtocol, - (void**)&This->protocol); - if(SUCCEEDED(hres)) - return S_OK; - - if(This->service_provider) { - hres = IServiceProvider_QueryService(This->service_provider, &IID_IInternetProtocol, - &IID_IInternetProtocol, (void**)&This->protocol); - if(SUCCEEDED(hres)) - return S_OK; - } - - hres = get_protocol_handler(url, NULL, &cf); - if(FAILED(hres)) - return hres; - - hres = IClassFactory_CreateInstance(cf, NULL, &IID_IInternetProtocol, (void**)&This->protocol); - IClassFactory_Release(cf); - - return hres; } static BOOL is_urlmon_protocol(LPCWSTR url) @@ -1110,20 +1320,20 @@ return FALSE; } -static HRESULT Binding_Create(LPCWSTR url, IBindCtx *pbc, REFIID riid, Binding **binding) +static HRESULT Binding_Create(IMoniker *mon, Binding *binding_ctx, LPCWSTR url, IBindCtx *pbc, + BOOL to_obj, REFIID riid, Binding **binding) { Binding *ret; - int len; HRESULT hres; - if(!IsEqualGUID(&IID_IStream, riid)) { + if(!to_obj && !IsEqualGUID(&IID_IStream, riid)) { FIXME("Unsupported riid %s\n", debugstr_guid(riid)); return E_NOTIMPL; } URLMON_LockModule(); - ret = urlmon_alloc(sizeof(Binding)); + ret = heap_alloc_zero(sizeof(Binding)); ret->lpBindingVtbl = &BindingVtbl; ret->lpInternetProtocolSinkVtbl = &InternetProtocolSinkVtbl; @@ -1132,23 +1342,24 @@ ret->ref = 1; - ret->callback = NULL; - ret->protocol = NULL; - ret->service_provider = NULL; - ret->stream = NULL; - ret->mime = NULL; - ret->url = NULL; + ret->to_object = to_obj; + ret->iid = *riid; ret->apartment_thread = GetCurrentThreadId(); ret->notif_hwnd = get_notif_hwnd(); - ret->report_mime = TRUE; - ret->continue_call = 0; - ret->request_locked = FALSE; + ret->report_mime = !binding_ctx; ret->download_state = BEFORE_DOWNLOAD; - ret->task_queue_head = ret->task_queue_tail = NULL; - - memset(&ret->bindinfo, 0, sizeof(BINDINFO)); + + if(to_obj) { + IBindCtx_AddRef(pbc); + ret->bctx = pbc; + } + + if(mon) { + IMoniker_AddRef(mon); + ret->mon = mon; + } + ret->bindinfo.cbSize = sizeof(BINDINFO); - ret->bindf = 0; InitializeCriticalSection(&ret->section); ret->section.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": Binding.section"); @@ -1163,11 +1374,16 @@ IBindStatusCallback_QueryInterface(ret->callback, &IID_IServiceProvider, (void**)&ret->service_provider); - hres = get_protocol(ret, url); - if(FAILED(hres)) { - WARN("Could not get protocol handler\n"); - IBinding_Release(BINDING(ret)); - return hres; + if(binding_ctx) { + ret->protocol = binding_ctx->protocol; + IInternetProtocol_AddRef(ret->protocol); + }else { + hres = create_binding_protocol(url, TRUE, &ret->protocol); + if(FAILED(hres)) { + WARN("Could not get protocol handler\n"); + IBinding_Release(BINDING(ret)); + return hres; + } } hres = IBindStatusCallback_GetBindInfo(ret->callback, &ret->bindf, &ret->bindinfo); @@ -1177,18 +1393,25 @@ return hres; } + TRACE("bindf %08x\n", ret->bindf); dump_BINDINFO(&ret->bindinfo); ret->bindf |= BINDF_FROMURLMON; + if(to_obj) + ret->bindinfo.dwOptions |= 0x100000; if(!is_urlmon_protocol(url)) ret->bindf |= BINDF_NEEDFILE; - len = strlenW(url)+1; - ret->url = urlmon_alloc(len*sizeof(WCHAR)); - memcpy(ret->url, url, len*sizeof(WCHAR)); - - ret->stream = create_stream(ret->protocol); + ret->url = heap_strdupW(url); + + if(binding_ctx) { + ret->stream = binding_ctx->stream; + IStream_AddRef(STREAM(ret->stream)); + ret->clipboard_format = binding_ctx->clipboard_format; + }else { + ret->stream = create_stream(ret->protocol); + } ret->stgmed.tymed = TYMED_ISTREAM; ret->stgmed.u.pstm = STREAM(ret->stream); ret->stgmed.pUnkForRelease = (IUnknown*)BINDING(ret); /* NOTE: Windows uses other IUnknown */ @@ -1197,41 +1420,44 @@ return S_OK; } -HRESULT start_binding(LPCWSTR url, IBindCtx *pbc, REFIID riid, void **ppv) +static HRESULT start_binding(IMoniker *mon, Binding *binding_ctx, LPCWSTR url, IBindCtx *pbc, + BOOL to_obj, REFIID riid, Binding **ret) { Binding *binding = NULL; HRESULT hres; MSG msg; - *ppv = NULL; - - hres = Binding_Create(url, pbc, riid, &binding); + hres = Binding_Create(mon, binding_ctx, url, pbc, to_obj, riid, &binding); if(FAILED(hres)) return hres; hres = IBindStatusCallback_OnStartBinding(binding->callback, 0, BINDING(binding)); if(FAILED(hres)) { WARN("OnStartBinding failed: %08x\n", hres); - IBindStatusCallback_OnStopBinding(binding->callback, 0x800c0008, NULL); + stop_binding(binding, 0x800c0008, NULL); IBinding_Release(BINDING(binding)); return hres; } - hres = IInternetProtocol_Start(binding->protocol, url, PROTSINK(binding), - BINDINF(binding), 0, 0); - - if(FAILED(hres)) { - WARN("Start failed: %08x\n", hres); - - IInternetProtocol_Terminate(binding->protocol, 0); - IBindStatusCallback_OnStopBinding(binding->callback, S_OK, NULL); - IBinding_Release(BINDING(binding)); - - return hres; + if(binding_ctx) { + set_binding_sink(binding->protocol, PROTSINK(binding)); + report_data(binding, 0, 0, 0); + }else { + hres = IInternetProtocol_Start(binding->protocol, url, PROTSINK(binding), + BINDINF(binding), 0, 0); + + TRACE("start ret %08x\n", hres); + + if(FAILED(hres)) { + stop_binding(binding, hres, NULL); + IBinding_Release(BINDING(binding)); + + return hres; + } } while(!(binding->bindf & BINDF_ASYNCHRONOUS) && - binding->download_state != END_DOWNLOAD) { + !(binding->state & BINDING_STOPPED)) { MsgWaitForMultipleObjects(0, NULL, FALSE, 5000, QS_POSTMESSAGE); while (PeekMessageW(&msg, binding->notif_hwnd, WM_USER, WM_USER+117, PM_REMOVE|PM_NOYIELD)) { TranslateMessage(&msg); @@ -1239,8 +1465,29 @@ } } - if(binding->stream->init_buf) { - if(binding->request_locked) + *ret = binding; + return S_OK; +} + +HRESULT bind_to_storage(LPCWSTR url, IBindCtx *pbc, REFIID riid, void **ppv) +{ + Binding *binding = NULL, *binding_ctx; + HRESULT hres; + + *ppv = NULL; + + binding_ctx = get_bctx_binding(pbc); + + hres = start_binding(NULL, binding_ctx, url, pbc, FALSE, riid, &binding); + if(binding_ctx) + IBinding_Release(BINDING(binding_ctx)); + if(FAILED(hres)) + return hres; + + if(binding->hres != S_OK) { + hres = SUCCEEDED(binding->hres) ? S_OK : binding->hres; + }else if(binding->stream->init_buf) { + if((binding->state & BINDING_STOPPED) && (binding->state & BINDING_LOCKED)) IInternetProtocol_UnlockRequest(binding->protocol); IStream_AddRef(STREAM(binding->stream)); @@ -1255,3 +1502,29 @@ return hres; } + +HRESULT bind_to_object(IMoniker *mon, LPCWSTR url, IBindCtx *pbc, REFIID riid, void **ppv) +{ + Binding *binding; + HRESULT hres; + + *ppv = NULL; + + hres = start_binding(mon, NULL, url, pbc, TRUE, riid, &binding); + if(FAILED(hres)) + return hres; + + if(binding->hres != S_OK) { + hres = SUCCEEDED(binding->hres) ? S_OK : binding->hres; + }else if(binding->bindf & BINDF_ASYNCHRONOUS) { + hres = MK_S_ASYNCHRONOUS; + }else { + *ppv = binding->obj; + IUnknown_AddRef(binding->obj); + hres = S_OK; + } + + IBinding_Release(BINDING(binding)); + + return hres; +} Modified: trunk/reactos/dll/win32/urlmon/bindprot.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/bindprot.… ============================================================================== --- trunk/reactos/dll/win32/urlmon/bindprot.c (original) +++ trunk/reactos/dll/win32/urlmon/bindprot.c Mon Jan 14 19:39:00 2008 @@ -16,17 +16,7 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> - -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" -#include "ole2.h" -#include "urlmon.h" #include "urlmon_main.h" - #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -35,6 +25,7 @@ const IInternetProtocolVtbl *lpInternetProtocolVtbl; const IInternetBindInfoVtbl *lpInternetBindInfoVtbl; const IInternetPriorityVtbl *lpInternetPriorityVtbl; + const IServiceProviderVtbl *lpServiceProviderVtbl; const IInternetProtocolSinkVtbl *lpInternetProtocolSinkVtbl; LONG ref; @@ -42,13 +33,18 @@ IInternetProtocol *protocol; IInternetBindInfo *bind_info; IInternetProtocolSink *protocol_sink; + IServiceProvider *service_provider; LONG priority; + + BOOL reported_result; + BOOL from_urlmon; } BindProtocol; #define PROTOCOL(x) ((IInternetProtocol*) &(x)->lpInternetProtocolVtbl) #define BINDINFO(x) ((IInternetBindInfo*) &(x)->lpInternetBindInfoVtbl) #define PRIORITY(x) ((IInternetPriority*) &(x)->lpInternetPriorityVtbl) +#define SERVPROV(x) ((IServiceProvider*) &(x)->lpServiceProviderVtbl) #define PROTSINK(x) ((IInternetProtocolSink*) &(x)->lpInternetProtocolSinkVtbl) #define PROTOCOL_THIS(iface) DEFINE_THIS(BindProtocol, InternetProtocol, iface) @@ -76,7 +72,8 @@ }else if(IsEqualGUID(&IID_IAuthenticate, riid)) { FIXME("(%p)->(IID_IAuthenticate %p)\n", This, ppv); }else if(IsEqualGUID(&IID_IServiceProvider, riid)) { - FIXME("(%p)->(IID_IServiceProvider %p)\n", This, ppv); + TRACE("(%p)->(IID_IServiceProvider %p)\n", This, ppv); + *ppv = SERVPROV(This); }else if(IsEqualGUID(&IID_IInternetProtocolSink, riid)) { TRACE("(%p)->(IID_IInternetProtocolSink %p)\n", This, ppv); *ppv = PROTSINK(This); @@ -111,10 +108,9 @@ IInternetProtocol_Release(This->protocol); if(This->bind_info) IInternetBindInfo_Release(This->bind_info); - if(This->protocol_sink) - IInternetProtocolSink_Release(This->protocol_sink); - - urlmon_free(This); + + set_binding_sink(PROTOCOL(This), NULL); + heap_free(This); URLMON_UnlockModule(); } @@ -157,16 +153,23 @@ if(FAILED(hres)) return hres; - hres = IClassFactory_CreateInstance(cf, (IUnknown*)BINDINFO(This), - &IID_IUnknown, (void**)&unk); - IClassFactory_Release(cf); - if(FAILED(hres)) - return hres; - - hres = IUnknown_QueryInterface(unk, &IID_IInternetProtocol, (void**)&protocol); - IUnknown_Release(unk); - if(FAILED(hres)) - return hres; + if(This->from_urlmon) { + hres = IClassFactory_CreateInstance(cf, NULL, &IID_IInternetProtocol, (void**)&protocol); + IClassFactory_Release(cf); + if(FAILED(hres)) + return hres; + }else { + hres = IClassFactory_CreateInstance(cf, (IUnknown*)BINDINFO(This), + &IID_IUnknown, (void**)&unk); + IClassFactory_Release(cf); + if(FAILED(hres)) + return hres; + + hres = IUnknown_QueryInterface(unk, &IID_IInternetProtocol, (void**)&protocol); + IUnknown_Release(unk); + if(FAILED(hres)) + return hres; + } } StringFromCLSID(&clsid, &clsid_str); @@ -178,8 +181,7 @@ IInternetBindInfo_AddRef(pOIBindInfo); This->bind_info = pOIBindInfo; - IInternetProtocolSink_AddRef(pOIProtSink); - This->protocol_sink = pOIProtSink; + set_binding_sink(PROTOCOL(This), pOIProtSink); hres = IInternetProtocol_QueryInterface(protocol, &IID_IInternetPriority, (void**)&priority); if(SUCCEEDED(hres)) { @@ -193,8 +195,10 @@ static HRESULT WINAPI BindProtocol_Continue(IInternetProtocol *iface, PROTOCOLDATA *pProtocolData) { BindProtocol *This = PROTOCOL_THIS(iface); - FIXME("(%p)->(%p)\n", This, pProtocolData); - return E_NOTIMPL; + + TRACE("(%p)->(%p)\n", This, pProtocolData); + + return IInternetProtocol_Continue(This->protocol, pProtocolData); } static HRESULT WINAPI BindProtocol_Abort(IInternetProtocol *iface, HRESULT hrReason, @@ -211,7 +215,18 @@ TRACE("(%p)->(%08x)\n", This, dwOptions); + if(!This->reported_result) + return E_FAIL; + IInternetProtocol_Terminate(This->protocol, 0); + + set_binding_sink(PROTOCOL(This), NULL); + + if(This->bind_info) { + IInternetBindInfo_Release(This->bind_info); + This->bind_info = NULL; + } + return S_OK; } @@ -255,15 +270,38 @@ static HRESULT WINAPI BindProtocol_LockRequest(IInternetProtocol *iface, DWORD dwOptions) { BindProtocol *This = PROTOCOL_THIS(iface); - FIXME("(%p)->(%08x)\n", This, dwOptions); - return E_NOTIMPL; + + TRACE("(%p)->(%08x)\n", This, dwOptions); + + return IInternetProtocol_LockRequest(This->protocol, dwOptions); } static HRESULT WINAPI BindProtocol_UnlockRequest(IInternetProtocol *iface) { BindProtocol *This = PROTOCOL_THIS(iface); - FIXME("(%p)\n", This); - return E_NOTIMPL; + + TRACE("(%p)\n", This); + + return IInternetProtocol_UnlockRequest(This->protocol); +} + +void set_binding_sink(IInternetProtocol *bind_protocol, IInternetProtocolSink *sink) +{ + BindProtocol *This = PROTOCOL_THIS(bind_protocol); + IInternetProtocolSink *prev_sink; + IServiceProvider *service_provider = NULL; + + if(sink) + IInternetProtocolSink_AddRef(sink); + prev_sink = InterlockedExchangePointer((void**)&This->protocol_sink, sink); + if(prev_sink) + IInternetProtocolSink_Release(prev_sink); + + if(sink) + IInternetProtocolSink_QueryInterface(sink, &IID_IServiceProvider, (void**)&service_provider); + service_provider = InterlockedExchangePointer((void**)&This->service_provider, service_provider); + if(service_provider) + IServiceProvider_Release(service_provider); } #undef PROTOCOL_THIS @@ -327,8 +365,10 @@ ULONG ulStringType, LPOLESTR *ppwzStr, ULONG cEl, ULONG *pcElFetched) { BindProtocol *This = BINDINFO_THIS(iface); - FIXME("(%p)->(%d %p %d %p)\n", This, ulStringType, ppwzStr, cEl, pcElFetched); - return E_NOTIMPL; + + TRACE("(%p)->(%d %p %d %p)\n", This, ulStringType, ppwzStr, cEl, pcElFetched); + + return IInternetBindInfo_GetBindString(This->bind_info, ulStringType, ppwzStr, cEl, pcElFetched); } #undef BINDFO_THIS @@ -395,34 +435,44 @@ #define PROTSINK_THIS(iface) DEFINE_THIS(BindProtocol, InternetProtocolSink, iface) -static HRESULT WINAPI InternetProtocolSink_QueryInterface(IInternetProtocolSink *iface, +static HRESULT WINAPI BPInternetProtocolSink_QueryInterface(IInternetProtocolSink *iface, REFIID riid, void **ppv) { BindProtocol *This = PROTSINK_THIS(iface); return IInternetProtocol_QueryInterface(PROTOCOL(This), riid, ppv); } -static ULONG WINAPI InternetProtocolSink_AddRef(IInternetProtocolSink *iface) +static ULONG WINAPI BPInternetProtocolSink_AddRef(IInternetProtocolSink *iface) { BindProtocol *This = PROTSINK_THIS(iface); return IInternetProtocol_AddRef(PROTOCOL(This)); } -static ULONG WINAPI InternetProtocolSink_Release(IInternetProtocolSink *iface) +static ULONG WINAPI BPInternetProtocolSink_Release(IInternetProtocolSink *iface) { BindProtocol *This = PROTSINK_THIS(iface); return IInternetProtocol_Release(PROTOCOL(This)); } -static HRESULT WINAPI InternetProtocolSink_Switch(IInternetProtocolSink *iface, +static HRESULT WINAPI BPInternetProtocolSink_Switch(IInternetProtocolSink *iface, PROTOCOLDATA *pProtocolData) { BindProtocol *This = PROTSINK_THIS(iface); - FIXME("(%p)->(%p)\n", This, pProtocolData); - return E_NOTIMPL; -} - -static HRESULT WINAPI InternetProtocolSink_ReportProgress(IInternetProtocolSink *iface, + + TRACE("(%p)->(%p)\n", This, pProtocolData); + + TRACE("flags %x state %x data %p cb %u\n", pProtocolData->grfFlags, pProtocolData->dwState, + pProtocolData->pData, pProtocolData->cbData); + + if(!This->protocol_sink) { + IInternetProtocol_Continue(This->protocol, pProtocolData); + return S_OK; + } + + return IInternetProtocolSink_Switch(This->protocol_sink, pProtocolData); +} + +static HRESULT WINAPI BPInternetProtocolSink_ReportProgress(IInternetProtocolSink *iface, ULONG ulStatusCode, LPCWSTR szStatusText) { BindProtocol *This = PROTSINK_THIS(iface); @@ -430,15 +480,24 @@ TRACE("(%p)->(%u %s)\n", This, ulStatusCode, debugstr_w(szStatusText)); switch(ulStatusCode) { + case BINDSTATUS_FINDINGRESOURCE: + case BINDSTATUS_CONNECTING: + case BINDSTATUS_BEGINDOWNLOADDATA: case BINDSTATUS_SENDINGREQUEST: - return IInternetProtocolSink_ReportProgress(This->protocol_sink, - ulStatusCode, NULL); case BINDSTATUS_CACHEFILENAMEAVAILABLE: + case BINDSTATUS_DIRECTBIND: + case BINDSTATUS_MIMETYPEAVAILABLE: + if(!This->protocol_sink) + return S_OK; return IInternetProtocolSink_ReportProgress(This->protocol_sink, ulStatusCode, szStatusText); + case BINDSTATUS_VERIFIEDMIMETYPEAVAILABLE: + if(!This->protocol_sink) + return S_OK; return IInternetProtocolSink_ReportProgress(This->protocol_sink, - BINDSTATUS_MIMETYPEAVAILABLE, szStatusText); + This->from_urlmon ? BINDSTATUS_VERIFIEDMIMETYPEAVAILABLE : BINDSTATUS_MIMETYPEAVAILABLE, + szStatusText); default: FIXME("unsupported ulStatusCode %u\n", ulStatusCode); } @@ -446,52 +505,101 @@ return E_NOTIMPL; } -static HRESULT WINAPI InternetProtocolSink_ReportData(IInternetProtocolSink *iface, +static HRESULT WINAPI BPInternetProtocolSink_ReportData(IInternetProtocolSink *iface, DWORD grfBSCF, ULONG ulProgress, ULONG ulProgressMax) { BindProtocol *This = PROTSINK_THIS(iface); TRACE("(%p)->(%d %u %u)\n", This, grfBSCF, ulProgress, ulProgressMax); - return S_OK; -} - -static HRESULT WINAPI InternetProtocolSink_ReportResult(IInternetProtocolSink *iface, + if(!This->protocol_sink) + return S_OK; + + return IInternetProtocolSink_ReportData(This->protocol_sink, grfBSCF, ulProgress, ulProgressMax); +} + +static HRESULT WINAPI BPInternetProtocolSink_ReportResult(IInternetProtocolSink *iface, HRESULT hrResult, DWORD dwError, LPCWSTR szResult) { BindProtocol *This = PROTSINK_THIS(iface); TRACE("(%p)->(%08x %d %s)\n", This, hrResult, dwError, debugstr_w(szResult)); + if(!This->protocol_sink) + return E_FAIL; + + This->reported_result = TRUE; + return IInternetProtocolSink_ReportResult(This->protocol_sink, hrResult, dwError, szResult); } #undef PROTSINK_THIS static const IInternetProtocolSinkVtbl InternetProtocolSinkVtbl = { - InternetProtocolSink_QueryInterface, - InternetProtocolSink_AddRef, - InternetProtocolSink_Release, - InternetProtocolSink_Switch, - InternetProtocolSink_ReportProgress, - InternetProtocolSink_ReportData, - InternetProtocolSink_ReportResult + BPInternetProtocolSink_QueryInterface, + BPInternetProtocolSink_AddRef, + BPInternetProtocolSink_Release, + BPInternetProtocolSink_Switch, + BPInternetProtocolSink_ReportProgress, + BPInternetProtocolSink_ReportData, + BPInternetProtocolSink_ReportResult }; -HRESULT create_binding_protocol(LPCWSTR url, IInternetProtocol **protocol) -{ - BindProtocol *ret = urlmon_alloc(sizeof(BindProtocol)); +#define SERVPROV_THIS(iface) DEFINE_THIS(BindProtocol, ServiceProvider, iface) + +static HRESULT WINAPI BPServiceProvider_QueryInterface(IServiceProvider *iface, + REFIID riid, void **ppv) +{ + BindProtocol *This = SERVPROV_THIS(iface); + return IInternetProtocol_QueryInterface(PROTOCOL(This), riid, ppv); +} + +static ULONG WINAPI BPServiceProvider_AddRef(IServiceProvider *iface) +{ + BindProtocol *This = SERVPROV_THIS(iface); + return IInternetProtocol_AddRef(PROTOCOL(This)); +} + +static ULONG WINAPI BPServiceProvider_Release(IServiceProvider *iface) +{ + BindProtocol *This = SERVPROV_THIS(iface); + return IInternetProtocol_Release(PROTOCOL(This)); +} + +static HRESULT WINAPI BPServiceProvider_QueryService(IServiceProvider *iface, + REFGUID guidService, REFIID riid, void **ppv) +{ + BindProtocol *This = SERVPROV_THIS(iface); + + TRACE("(%p)->(%s %s %p)\n", This, debugstr_guid(guidService), debugstr_guid(riid), ppv); + + if(!This->service_provider) + return E_NOINTERFACE; + + return IServiceProvider_QueryService(This->service_provider, guidService, riid, ppv); +} + +#undef SERVPROV_THIS + +static const IServiceProviderVtbl ServiceProviderVtbl = { + BPServiceProvider_QueryInterface, + BPServiceProvider_AddRef, + BPServiceProvider_Release, + BPServiceProvider_QueryService +}; + +HRESULT create_binding_protocol(LPCWSTR url, BOOL from_urlmon, IInternetProtocol **protocol) +{ + BindProtocol *ret = heap_alloc_zero(sizeof(BindProtocol)); ret->lpInternetProtocolVtbl = &BindProtocolVtbl; ret->lpInternetBindInfoVtbl = &InternetBindInfoVtbl; ret->lpInternetPriorityVtbl = &InternetPriorityVtbl; + ret->lpServiceProviderVtbl = &ServiceProviderVtbl; ret->lpInternetProtocolSinkVtbl = &InternetProtocolSinkVtbl; ret->ref = 1; - ret->protocol = NULL; - ret->bind_info = NULL; - ret->protocol_sink = NULL; - ret->priority = 0; + ret->from_urlmon = from_urlmon; URLMON_LockModule(); Modified: trunk/reactos/dll/win32/urlmon/file.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/file.c?re… ============================================================================== --- trunk/reactos/dll/win32/urlmon/file.c (original) +++ trunk/reactos/dll/win32/urlmon/file.c Mon Jan 14 19:39:00 2008 @@ -16,17 +16,7 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> - -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" -#include "ole2.h" -#include "urlmon.h" #include "urlmon_main.h" - #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -92,7 +82,7 @@ if(!ref) { if(This->file) CloseHandle(This->file); - urlmon_free(This); + heap_free(This); URLMON_UnlockModule(); } @@ -135,10 +125,10 @@ return MK_E_SYNTAX; len = lstrlenW(szUrl)+16; - url = urlmon_alloc(len*sizeof(WCHAR)); + url = heap_alloc(len*sizeof(WCHAR)); hres = CoInternetParseUrl(szUrl, PARSE_ENCODE, 0, url, len, &len, 0); if(FAILED(hres)) { - urlmon_free(url); + heap_free(url); return hres; } @@ -163,7 +153,7 @@ This->file = NULL; IInternetProtocolSink_ReportResult(pOIProtSink, INET_E_RESOURCE_NOT_FOUND, GetLastError(), NULL); - urlmon_free(url); + heap_free(url); return INET_E_RESOURCE_NOT_FOUND; } @@ -180,7 +170,7 @@ } } - urlmon_free(url); + heap_free(url); if(GetFileSizeEx(This->file, &size)) IInternetProtocolSink_ReportData(pOIProtSink, @@ -353,7 +343,7 @@ URLMON_LockModule(); - ret = urlmon_alloc(sizeof(FileProtocol)); + ret = heap_alloc(sizeof(FileProtocol)); ret->lpInternetProtocolVtbl = &FileProtocolVtbl; ret->lpInternetPriorityVtbl = &FilePriorityVtbl; Modified: trunk/reactos/dll/win32/urlmon/format.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/format.c?… ============================================================================== --- trunk/reactos/dll/win32/urlmon/format.c (original) +++ trunk/reactos/dll/win32/urlmon/format.c Mon Jan 14 19:39:00 2008 @@ -16,17 +16,7 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> - -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" -#include "ole2.h" -#include "urlmon.h" #include "urlmon_main.h" - #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -54,6 +44,7 @@ *ppv = NULL; if(IsEqualGUID(&IID_IUnknown, riid) || IsEqualGUID(&IID_IEnumFORMATETC, riid)) { + IEnumFORMATETC_AddRef(iface); *ppv = iface; return S_OK; } @@ -78,8 +69,8 @@ TRACE("(%p) ref=%d\n", This, ref); if(!ref) { - urlmon_free(This->fetc); - urlmon_free(This); + heap_free(This->fetc); + heap_free(This); URLMON_UnlockModule(); } @@ -160,7 +151,7 @@ static IEnumFORMATETC *EnumFORMATETC_Create(UINT cfmtetc, const FORMATETC *rgfmtetc, UINT it) { - EnumFORMATETC *ret = urlmon_alloc(sizeof(EnumFORMATETC)); + EnumFORMATETC *ret = heap_alloc(sizeof(EnumFORMATETC)); URLMON_LockModule(); @@ -169,7 +160,7 @@ ret->it = it; ret->fetc_cnt = cfmtetc; - ret->fetc = urlmon_alloc(cfmtetc*sizeof(FORMATETC)); + ret->fetc = heap_alloc(cfmtetc*sizeof(FORMATETC)); memcpy(ret->fetc, rgfmtetc, cfmtetc*sizeof(FORMATETC)); return (IEnumFORMATETC*)ret; Modified: trunk/reactos/dll/win32/urlmon/ftp.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/ftp.c?rev… ============================================================================== --- trunk/reactos/dll/win32/urlmon/ftp.c (original) +++ trunk/reactos/dll/win32/urlmon/ftp.c Mon Jan 14 19:39:00 2008 @@ -16,17 +16,7 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> - -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" -#include "ole2.h" -#include "urlmon.h" #include "urlmon_main.h" - #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -81,7 +71,7 @@ TRACE("(%p) ref=%d\n", This, ref); if(!ref) { - urlmon_free(This); + heap_free(This); URLMON_UnlockModule(); } @@ -191,7 +181,7 @@ URLMON_LockModule(); - ret = urlmon_alloc(sizeof(FtpProtocol)); + ret = heap_alloc(sizeof(FtpProtocol)); ret->lpInternetProtocolVtbl = &FtpProtocolVtbl; ret->ref = 1; Modified: trunk/reactos/dll/win32/urlmon/http.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/http.c?re… ============================================================================== --- trunk/reactos/dll/win32/urlmon/http.c (original) +++ trunk/reactos/dll/win32/urlmon/http.c Mon Jan 14 19:39:00 2008 @@ -22,21 +22,10 @@ * - Handle redirects as native. */ -#include <stdarg.h> - -#define COBJMACROS -#define NONAMELESSUNION - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" -#include "ole2.h" -#include "urlmon.h" +#include "urlmon_main.h" #include "wininet.h" -#include "urlmon_main.h" #include "wine/debug.h" -#include "wine/unicode.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -159,7 +148,7 @@ if (This->full_header) { if (This->full_header != wszHeaders) - urlmon_free(This->full_header); + heap_free(This->full_header); This->full_header = 0; } This->flags = 0; @@ -234,7 +223,7 @@ { LPWSTR ret = NULL; if (string && - (ret = urlmon_alloc((len+1)*sizeof(WCHAR))) != NULL) + (ret = heap_alloc((len+1)*sizeof(WCHAR))) != NULL) { memcpy(ret, string, len*sizeof(WCHAR)); ret[len] = 0; @@ -296,7 +285,7 @@ if(!ref) { HTTPPROTOCOL_Close(This); - urlmon_free(This); + heap_free(This); URLMON_UnlockModule(); } @@ -317,7 +306,7 @@ LPWSTR host = 0, path = 0, user = 0, pass = 0, addl_header = 0, post_cookie = 0, optional = 0; BYTE security_id[512]; - LPOLESTR user_agent, accept_mimes[257]; + LPOLESTR user_agent = NULL, accept_mimes[257]; HRESULT hres; static const WCHAR wszHttp[] = {'h','t','t','p',':'}; @@ -378,7 +367,7 @@ { WARN("ObtainUserAgentString failed: %08x\n", hres); } - else if (!(user_agenta = urlmon_alloc(len*sizeof(CHAR)))) + else if (!(user_agenta = heap_alloc(len*sizeof(CHAR)))) { WARN("Out of memory\n"); } @@ -393,7 +382,7 @@ else MultiByteToWideChar(CP_ACP, 0, user_agenta, -1, user_agent, len*sizeof(WCHAR)); } - urlmon_free(user_agenta); + heap_free(user_agenta); } This->internet = InternetOpenW(user_agent, 0, NULL, NULL, INTERNET_FLAG_ASYNC); @@ -472,7 +461,7 @@ else { int len_addl_header = lstrlenW(addl_header); - This->full_header = urlmon_alloc(len_addl_header*sizeof(WCHAR)+sizeof(wszHeaders)); + This->full_header = heap_alloc(len_addl_header*sizeof(WCHAR)+sizeof(wszHeaders)); if (!This->full_header) { WARN("Out of memory\n"); @@ -556,10 +545,10 @@ CoTaskMemFree(accept_mimes[num++]); CoTaskMemFree(user_agent); - urlmon_free(pass); - urlmon_free(user); - urlmon_free(path); - urlmon_free(host); + heap_free(pass); + heap_free(user); + heap_free(path); + heap_free(host); return hres; } @@ -609,7 +598,7 @@ if ((!HttpQueryInfoW(This->request, HTTP_QUERY_RAW_HEADERS_CRLF, response_headers, &len, NULL) && GetLastError() != ERROR_INSUFFICIENT_BUFFER) || - !(response_headers = urlmon_alloc(len)) || + !(response_headers = heap_alloc(len)) || !HttpQueryInfoW(This->request, HTTP_QUERY_RAW_HEADERS_CRLF, response_headers, &len, NULL)) { @@ -630,7 +619,7 @@ len = 0; if ((!HttpQueryInfoW(This->request, HTTP_QUERY_CONTENT_TYPE, content_type, &len, NULL) && GetLastError() != ERROR_INSUFFICIENT_BUFFER) || - !(content_type = urlmon_alloc(len)) || + !(content_type = heap_alloc(len)) || !HttpQueryInfoW(This->request, HTTP_QUERY_CONTENT_TYPE, content_type, &len, NULL)) { WARN("HttpQueryInfo failed: %d\n", GetLastError()); @@ -656,7 +645,7 @@ len = 0; if ((!HttpQueryInfoW(This->request, HTTP_QUERY_CONTENT_LENGTH, content_length, &len, NULL) && GetLastError() != ERROR_INSUFFICIENT_BUFFER) || - !(content_length = urlmon_alloc(len)) || + !(content_length = heap_alloc(len)) || !HttpQueryInfoW(This->request, HTTP_QUERY_CONTENT_LENGTH, content_length, &len, NULL)) { WARN("HttpQueryInfo failed: %d\n", GetLastError()); @@ -693,9 +682,9 @@ } done: - urlmon_free(response_headers); - urlmon_free(content_type); - urlmon_free(content_length); + heap_free(response_headers); + heap_free(content_type); + heap_free(content_length); /* Returns S_OK on native */ return S_OK; @@ -925,7 +914,7 @@ URLMON_LockModule(); - ret = urlmon_alloc(sizeof(HttpProtocol)); + ret = heap_alloc(sizeof(HttpProtocol)); ret->lpInternetProtocolVtbl = &HttpProtocolVtbl; ret->lpInternetPriorityVtbl = &HttpPriorityVtbl; Modified: trunk/reactos/dll/win32/urlmon/internet.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/internet.… ============================================================================== --- trunk/reactos/dll/win32/urlmon/internet.c (original) +++ trunk/reactos/dll/win32/urlmon/internet.c Mon Jan 14 19:39:00 2008 @@ -16,21 +16,11 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> - -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" +#include "urlmon_main.h" #include "winreg.h" #include "shlwapi.h" -#include "ole2.h" -#include "urlmon.h" -#include "urlmon_main.h" #include "wine/debug.h" -#include "wine/unicode.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -264,3 +254,47 @@ return UrlCompareW(pwzUrl1, pwzUrl2, dwCompareFlags) ? S_FALSE : S_OK; } + +/*********************************************************************** + * CoInternetQueryInfo (URLMON.@) + * + * Retrieves information relevant to a specified URL + * + */ +HRESULT WINAPI CoInternetQueryInfo(LPCWSTR pwzUrl, QUERYOPTION QueryOption, + DWORD dwQueryFlags, LPVOID pvBuffer, DWORD cbBuffer, DWORD *pcbBuffer, + DWORD dwReserved) +{ + IInternetProtocolInfo *protocol_info; + HRESULT hres; + + TRACE("(%s, %x, %x, %p, %x, %p, %x): stub\n", debugstr_w(pwzUrl), + QueryOption, dwQueryFlags, pvBuffer, cbBuffer, pcbBuffer, dwReserved); + + protocol_info = get_protocol_info(pwzUrl); + + if(protocol_info) { + hres = IInternetProtocolInfo_QueryInfo(protocol_info, pwzUrl, QueryOption, dwQueryFlags, + pvBuffer, cbBuffer, pcbBuffer, dwReserved); + IInternetProtocolInfo_Release(protocol_info); + + return SUCCEEDED(hres) ? hres : E_FAIL; + } + + switch(QueryOption) { + case QUERY_USES_NETWORK: + if(!pvBuffer || cbBuffer < sizeof(DWORD)) + return E_FAIL; + + *(DWORD*)pvBuffer = 0; + if(pcbBuffer) + *pcbBuffer = sizeof(DWORD); + break; + + default: + FIXME("Not supported option %d\n", QueryOption); + return E_NOTIMPL; + } + + return S_OK; +} Modified: trunk/reactos/dll/win32/urlmon/mk.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/mk.c?rev=… ============================================================================== --- trunk/reactos/dll/win32/urlmon/mk.c (original) +++ trunk/reactos/dll/win32/urlmon/mk.c Mon Jan 14 19:39:00 2008 @@ -16,19 +16,8 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> - -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" -#include "ole2.h" -#include "urlmon.h" #include "urlmon_main.h" - #include "wine/debug.h" -#include "wine/unicode.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -88,7 +77,7 @@ if(This->stream) IStream_Release(This->stream); - urlmon_free(This); + heap_free(This); URLMON_UnlockModule(); } @@ -153,11 +142,11 @@ if(!ptr) return report_result(pOIProtSink, INET_E_RESOURCE_NOT_FOUND, ERROR_INVALID_PARAMETER); - progid = urlmon_alloc((ptr-ptr2+1)*sizeof(WCHAR)); + progid = heap_alloc((ptr-ptr2+1)*sizeof(WCHAR)); memcpy(progid, ptr2, (ptr-ptr2)*sizeof(WCHAR)); progid[ptr-ptr2] = 0; hres = CLSIDFromProgID(progid, &clsid); - urlmon_free(progid); + heap_free(progid); if(FAILED(hres)) return report_result(pOIProtSink, INET_E_RESOURCE_NOT_FOUND, ERROR_INVALID_PARAMETER); @@ -169,10 +158,10 @@ } len = strlenW(--ptr2); - display_name = urlmon_alloc((len+1)*sizeof(WCHAR)); + display_name = heap_alloc((len+1)*sizeof(WCHAR)); memcpy(display_name, ptr2, (len+1)*sizeof(WCHAR)); hres = IParseDisplayName_ParseDisplayName(pdn, NULL /* FIXME */, display_name, &eaten, &mon); - urlmon_free(display_name); + heap_free(display_name); IParseDisplayName_Release(pdn); if(FAILED(hres)) { WARN("ParseDisplayName failed: %08x\n", hres); @@ -307,7 +296,7 @@ URLMON_LockModule(); - ret = urlmon_alloc(sizeof(MkProtocol)); + ret = heap_alloc(sizeof(MkProtocol)); ret->lpInternetProtocolVtbl = &MkProtocolVtbl; ret->ref = 1; Modified: trunk/reactos/dll/win32/urlmon/regsvr.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/regsvr.c?… ============================================================================== --- trunk/reactos/dll/win32/urlmon/regsvr.c (original) +++ trunk/reactos/dll/win32/urlmon/regsvr.c Mon Jan 14 19:39:00 2008 @@ -18,25 +18,14 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> #include <stdio.h> -#include <string.h> - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" -#include "wingdi.h" + +#include "urlmon_main.h" + #include "winreg.h" -#include "winerror.h" #include "advpub.h" -#include "objbase.h" - -#include "urlmon.h" - #include "wine/debug.h" - -#include "urlmon_main.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -550,7 +539,7 @@ INF_SET_CLSID(MkProtocol); for(i = 0; i < sizeof(pse)/sizeof(pse[0]); i++) { - pse[i].pszValue = urlmon_alloc(39); + pse[i].pszValue = heap_alloc(39); sprintf(pse[i].pszValue, "{%08X-%04X-%04X-%02X%02X-%02X%02X%02X%02X%02X%02X}", clsids[i]->Data1, clsids[i]->Data2, clsids[i]->Data3, clsids[i]->Data4[0], clsids[i]->Data4[1], clsids[i]->Data4[2], clsids[i]->Data4[3], clsids[i]->Data4[4], @@ -566,7 +555,7 @@ hres = pRegInstall(URLMON_hInstance, doregister ? "RegisterDll" : "UnregisterDll", &strtable); for(i=0; i < sizeof(pse)/sizeof(pse[0]); i++) - urlmon_free(pse[i].pszValue); + heap_free(pse[i].pszValue); return hres; } Modified: trunk/reactos/dll/win32/urlmon/sec_mgr.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/sec_mgr.c… ============================================================================== --- trunk/reactos/dll/win32/urlmon/sec_mgr.c (original) +++ trunk/reactos/dll/win32/urlmon/sec_mgr.c Mon Jan 14 19:39:00 2008 @@ -19,20 +19,12 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> #include <stdio.h> -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" +#include "urlmon_main.h" #include "winreg.h" + #include "wine/debug.h" -#include "ole2.h" -#include "wine/unicode.h" -#include "urlmon.h" -#include "urlmon_main.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -173,7 +165,7 @@ if(This->custom_manager) IInternetSecurityManager_Release(This->custom_manager); - urlmon_free(This); + heap_free(This); URLMON_UnlockModule(); } @@ -260,7 +252,7 @@ FIXME("not supported flags: %08x\n", dwFlags); size = (strlenW(pwszUrl)+16) * sizeof(WCHAR); - url = urlmon_alloc(size); + url = heap_alloc(size); hres = CoInternetParseUrl(pwszUrl, PARSE_SECURITY_URL, 0, url, size/sizeof(WCHAR), &size, 0); if(FAILED(hres)) @@ -268,7 +260,7 @@ hres = map_url_to_zone(url, pdwZone); - urlmon_free(url); + heap_free(url); return hres; } @@ -300,7 +292,7 @@ FIXME("dwReserved is not supported\n"); len = strlenW(pwszUrl)+1; - buf = urlmon_alloc((len+16)*sizeof(WCHAR)); + buf = heap_alloc((len+16)*sizeof(WCHAR)); hres = CoInternetParseUrl(pwszUrl, PARSE_SECURITY_URL, 0, buf, len, &size, 0); if(FAILED(hres)) @@ -308,7 +300,7 @@ hres = map_url_to_zone(buf, &zone); if(FAILED(hres)) { - urlmon_free(buf); + heap_free(buf); return hres == 0x80041001 ? E_INVALIDARG : hres; } @@ -318,7 +310,7 @@ static const BYTE secidFile[] = {'f','i','l','e',':'}; - urlmon_free(buf); + heap_free(buf); if(*pcbSecurityId < sizeof(secidFile)+sizeof(zone)) return HRESULT_FROM_WIN32(ERROR_INSUFFICIENT_BUFFER); @@ -344,12 +336,12 @@ len = WideCharToMultiByte(CP_ACP, 0, buf, -1, NULL, 0, NULL, NULL)-1; if(len+sizeof(DWORD) > *pcbSecurityId) { - urlmon_free(buf); + heap_free(buf); return HRESULT_FROM_WIN32(ERROR_INSUFFICIENT_BUFFER); } WideCharToMultiByte(CP_ACP, 0, buf, -1, (LPSTR)pbSecurityId, -1, NULL, NULL); - urlmon_free(buf); + heap_free(buf); *(DWORD*)(pbSecurityId+len) = zone; @@ -464,7 +456,7 @@ SecManagerImpl *This; TRACE("(%p,%p)\n",pUnkOuter,ppobj); - This = urlmon_alloc(sizeof(*This)); + This = heap_alloc(sizeof(*This)); /* Initialize the virtual function table. */ This->lpInternetSecurityManagerVtbl = &VT_SecManagerImpl; @@ -577,7 +569,7 @@ TRACE("(%p)->(ref before=%u)\n",This, refCount + 1); if(!refCount) { - urlmon_free(This); + heap_free(This); URLMON_UnlockModule(); } @@ -790,7 +782,7 @@ HRESULT ZoneMgrImpl_Construct(IUnknown *pUnkOuter, LPVOID *ppobj) { - ZoneMgrImpl* ret = urlmon_alloc(sizeof(ZoneMgrImpl)); + ZoneMgrImpl* ret = heap_alloc(sizeof(ZoneMgrImpl)); TRACE("(%p %p)\n", pUnkOuter, ppobj); ret->lpVtbl = &ZoneMgrImplVtbl; Modified: trunk/reactos/dll/win32/urlmon/session.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/session.c… ============================================================================== --- trunk/reactos/dll/win32/urlmon/session.c (original) +++ trunk/reactos/dll/win32/urlmon/session.c Mon Jan 14 19:39:00 2008 @@ -16,20 +16,10 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> - -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" -#include "winuser.h" +#include "urlmon_main.h" #include "winreg.h" -#include "ole2.h" -#include "urlmon.h" -#include "urlmon_main.h" #include "wine/debug.h" -#include "wine/unicode.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -41,7 +31,16 @@ struct name_space *next; } name_space; +typedef struct mime_filter { + IClassFactory *cf; + CLSID clsid; + LPWSTR mime; + + struct mime_filter *next; +} mime_filter; + static name_space *name_space_list = NULL; +static mime_filter *mime_filter_list = NULL; static name_space *find_name_space(LPCWSTR protocol) { @@ -68,12 +67,12 @@ {'P','R','O','T','O','C','O','L','S','\\','H','a','n','d','l','e','r','\\'}; static const WCHAR wszCLSID[] = {'C','L','S','I','D',0}; - wszKey = urlmon_alloc(sizeof(wszProtocolsKey)+(schema_len+1)*sizeof(WCHAR)); + wszKey = heap_alloc(sizeof(wszProtocolsKey)+(schema_len+1)*sizeof(WCHAR)); memcpy(wszKey, wszProtocolsKey, sizeof(wszProtocolsKey)); memcpy(wszKey + sizeof(wszProtocolsKey)/sizeof(WCHAR), schema, (schema_len+1)*sizeof(WCHAR)); res = RegOpenKeyW(HKEY_CLASSES_ROOT, wszKey, &hkey); - urlmon_free(wszKey); + heap_free(wszKey); if(res != ERROR_SUCCESS) { TRACE("Could not open protocol handler key\n"); return E_FAIL; @@ -194,7 +193,6 @@ const LPCWSTR *ppwzPatterns, DWORD dwReserved) { name_space *new_name_space; - int size; TRACE("(%p %s %s %d %p %d)\n", pCF, debugstr_guid(rclsid), debugstr_w(pwzProtocol), cPatterns, ppwzPatterns, dwReserved); @@ -207,15 +205,12 @@ if(!pCF || !pwzProtocol) return E_INVALIDARG; - new_name_space = urlmon_alloc(sizeof(name_space)); - - size = (strlenW(pwzProtocol)+1)*sizeof(WCHAR); - new_name_space->protocol = urlmon_alloc(size); - memcpy(new_name_space->protocol, pwzProtocol, size); + new_name_space = heap_alloc(sizeof(name_space)); IClassFactory_AddRef(pCF); new_name_space->cf = pCF; new_name_space->clsid = *rclsid; + new_name_space->protocol = heap_strdupW(pwzProtocol); new_name_space->next = name_space_list; name_space_list = new_name_space; @@ -247,8 +242,8 @@ name_space_list = iter->next; IClassFactory_Release(iter->cf); - urlmon_free(iter->protocol); - urlmon_free(iter); + heap_free(iter->protocol); + heap_free(iter); return S_OK; } @@ -256,15 +251,49 @@ static HRESULT WINAPI InternetSession_RegisterMimeFilter(IInternetSession *iface, IClassFactory *pCF, REFCLSID rclsid, LPCWSTR pwzType) { - FIXME("(%p %s %s)\n", pCF, debugstr_guid(rclsid), debugstr_w(pwzType)); - return E_NOTIMPL; + mime_filter *filter; + + TRACE("(%p %s %s)\n", pCF, debugstr_guid(rclsid), debugstr_w(pwzType)); + + filter = heap_alloc(sizeof(mime_filter)); + + IClassFactory_AddRef(pCF); + filter->cf = pCF; + filter->clsid = *rclsid; + filter->mime = heap_strdupW(pwzType); + + filter->next = mime_filter_list; + mime_filter_list = filter; + + return S_OK; } static HRESULT WINAPI InternetSession_UnregisterMimeFilter(IInternetSession *iface, IClassFactory *pCF, LPCWSTR pwzType) { - FIXME("(%p %s)\n", pCF, debugstr_w(pwzType)); - return E_NOTIMPL; + mime_filter *iter, *prev = NULL; + + TRACE("(%p %s)\n", pCF, debugstr_w(pwzType)); + + for(iter = mime_filter_list; iter; iter = iter->next) { + if(iter->cf == pCF && !strcmpW(iter->mime, pwzType)) + break; + prev = iter; + } + + if(!iter) + return S_OK; + + if(prev) + prev->next = iter->next; + else + mime_filter_list = iter->next; + + IClassFactory_Release(iter->cf); + heap_free(iter->mime); + heap_free(iter); + + return S_OK; } static HRESULT WINAPI InternetSession_CreateBinding(IInternetSession *iface, @@ -277,7 +306,7 @@ if(pBC || pUnkOuter || ppUnk || dwOption) FIXME("Unsupported arguments\n"); - return create_binding_protocol(szUrl, ppOInetProt); + return create_binding_protocol(szUrl, FALSE, ppOInetProt); } static HRESULT WINAPI InternetSession_SetSessionOption(IInternetSession *iface, Modified: trunk/reactos/dll/win32/urlmon/umon.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/umon.c?re… ============================================================================== --- trunk/reactos/dll/win32/urlmon/umon.c (original) +++ trunk/reactos/dll/win32/urlmon/umon.c Mon Jan 14 19:39:00 2008 @@ -20,26 +20,16 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> #include <stdio.h> -#define COBJMACROS -#define NONAMELESSUNION -#define NONAMELESSSTRUCT - -#include "windef.h" -#include "winbase.h" +#include "urlmon_main.h" + #include "winreg.h" #include "winternl.h" -#include "winuser.h" -#include "objbase.h" -#include "wine/debug.h" -#include "wine/unicode.h" -#include "ole2.h" -#include "urlmon.h" #include "wininet.h" #include "shlwapi.h" -#include "urlmon_main.h" + +#include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -101,7 +91,7 @@ TRACE("(%p) ref=%d\n",This, ref); if(!ref) { - urlmon_free(This->URLName); + heap_free(This->URLName); if (This->hCacheFile) CloseHandle(This->hCacheFile); if (This->pstrCache) @@ -112,7 +102,7 @@ if (This->pbscb) IBindStatusCallback_Release(This->pbscb); - urlmon_free(This); + heap_free(This); URLMON_UnlockModule(); } @@ -356,8 +346,8 @@ /* destroy the object if there's no more reference on it */ if (!refCount) { - urlmon_free(This->URLName); - urlmon_free(This); + heap_free(This->URLName); + heap_free(This); URLMON_UnlockModule(); } @@ -420,8 +410,8 @@ res = IStream_Read(pStm, &size, sizeof(ULONG), &got); if(SUCCEEDED(res)) { if(got == sizeof(ULONG)) { - urlmon_free(This->URLName); - This->URLName = urlmon_alloc(size); + heap_free(This->URLName); + This->URLName = heap_alloc(size); if(!This->URLName) res = E_OUTOFMEMORY; else { @@ -483,16 +473,21 @@ IBindCtx* pbc, IMoniker* pmkToLeft, REFIID riid, - VOID** ppvResult) -{ - URLMonikerImpl *This = (URLMonikerImpl *)iface; - - *ppvResult=0; - - FIXME("(%p)->(%p,%p,%s,%p): stub\n",This,pbc,pmkToLeft,debugstr_guid(riid), - ppvResult); - - return E_NOTIMPL; + VOID** ppv) +{ + URLMonikerImpl *This = (URLMonikerImpl *)iface; + IRunningObjectTable *obj_tbl; + HRESULT hres; + + TRACE("(%p)->(%p,%p,%s,%p): stub\n", This, pbc, pmkToLeft, debugstr_guid(riid), ppv); + + hres = IBindCtx_GetRunningObjectTable(pbc, &obj_tbl); + if(SUCCEEDED(hres)) { + FIXME("use running object table\n"); + IRunningObjectTable_Release(obj_tbl); + } + + return bind_to_object(iface, This->URLName, pbc, riid, ppv); } /****************************************************************************** @@ -517,13 +512,13 @@ return E_NOTIMPL; } - bind = urlmon_alloc_zero(sizeof(Binding)); + bind = heap_alloc_zero(sizeof(Binding)); bind->lpVtbl = &BindingVtbl; bind->ref = 1; URLMON_LockModule(); len = lstrlenW(URLName)+1; - bind->URLName = urlmon_alloc(len*sizeof(WCHAR)); + bind->URLName = heap_alloc(len*sizeof(WCHAR)); memcpy(bind->URLName, URLName, len*sizeof(WCHAR)); hres = UMCreateStreamOnCacheFile(bind->URLName, 0, szFileName, &bind->hCacheFile, &bind->pstrCache); @@ -561,15 +556,15 @@ url.dwStructSize = sizeof(url); url.dwSchemeLength = url.dwHostNameLength = url.dwUrlPathLength = url.dwUserNameLength = url.dwPasswordLength = 1; InternetCrackUrlW(URLName, 0, ICU_ESCAPE, &url); - host = urlmon_alloc((url.dwHostNameLength + 1) * sizeof(WCHAR)); + host = heap_alloc((url.dwHostNameLength + 1) * sizeof(WCHAR)); memcpy(host, url.lpszHostName, url.dwHostNameLength * sizeof(WCHAR)); host[url.dwHostNameLength] = '\0'; - path = urlmon_alloc((url.dwUrlPathLength + 1) * sizeof(WCHAR)); + path = heap_alloc((url.dwUrlPathLength + 1) * sizeof(WCHAR)); memcpy(path, url.lpszUrlPath, url.dwUrlPathLength * sizeof(WCHAR)); path[url.dwUrlPathLength] = '\0'; if (url.dwUserNameLength) { - user = urlmon_alloc(((url.dwUserNameLength + 1) * sizeof(WCHAR))); + user = heap_alloc(((url.dwUserNameLength + 1) * sizeof(WCHAR))); memcpy(user, url.lpszUserName, url.dwUserNameLength * sizeof(WCHAR)); user[url.dwUserNameLength] = 0; } @@ -579,7 +574,7 @@ } if (url.dwPasswordLength) { - pass = urlmon_alloc(((url.dwPasswordLength + 1) * sizeof(WCHAR))); + pass = heap_alloc(((url.dwPasswordLength + 1) * sizeof(WCHAR))); memcpy(pass, url.lpszPassword, url.dwPasswordLength * sizeof(WCHAR)); pass[url.dwPasswordLength] = 0; } @@ -711,10 +706,10 @@ Binding_FinishedDownload(bind, hres); Binding_CloseCacheDownload(bind); - urlmon_free(user); - urlmon_free(pass); - urlmon_free(path); - urlmon_free(host); + heap_free(user); + heap_free(pass); + heap_free(path); + heap_free(host); } } } @@ -753,7 +748,7 @@ TRACE("(%p)->(%p %p %s %p)\n", This, pbc, pmkToLeft, debugstr_guid(riid), ppvObject); - return start_binding(This->URLName, pbc, riid, ppvObject); + return bind_to_storage(This->URLName, pbc, riid, ppvObject); } /****************************************************************************** @@ -1042,7 +1037,7 @@ This->lpvtbl = &VT_URLMonikerImpl; This->ref = 0; - This->URLName = urlmon_alloc(INTERNET_MAX_URL_LENGTH*sizeof(WCHAR)); + This->URLName = heap_alloc(INTERNET_MAX_URL_LENGTH*sizeof(WCHAR)); if(lpszLeftURLName) hres = CoInternetCombineUrl(lpszLeftURLName, lpszURLName, URL_FILE_USE_PATHURL, @@ -1052,14 +1047,14 @@ This->URLName, INTERNET_MAX_URL_LENGTH, &sizeStr, 0); if(FAILED(hres)) { - urlmon_free(This->URLName); + heap_free(This->URLName); return hres; } URLMON_LockModule(); if(sizeStr != INTERNET_MAX_URL_LENGTH) - This->URLName = urlmon_realloc(This->URLName, (sizeStr+1)*sizeof(WCHAR)); + This->URLName = heap_realloc(This->URLName, (sizeStr+1)*sizeof(WCHAR)); TRACE("URLName = %s\n", debugstr_w(This->URLName)); @@ -1092,7 +1087,7 @@ if (dwFlags & URL_MK_UNIFORM) FIXME("ignoring flag URL_MK_UNIFORM\n"); - if(!(obj = urlmon_alloc(sizeof(*obj)))) + if(!(obj = heap_alloc(sizeof(*obj)))) return E_OUTOFMEMORY; if(pmkContext) { @@ -1111,7 +1106,7 @@ if(SUCCEEDED(hres)) hres = URLMonikerImpl_QueryInterface((IMoniker*)obj, &IID_IMoniker, (void**)ppmk); else - urlmon_free(obj); + heap_free(obj); return hres; } @@ -1136,26 +1131,6 @@ } /*********************************************************************** - * CoInternetQueryInfo (URLMON.@) - * - * Retrieves information relevant to a specified URL - * - * RETURNS - * S_OK success - * S_FALSE buffer too small - * INET_E_QUERYOPTIONUNKNOWN invalid option - * - */ -HRESULT WINAPI CoInternetQueryInfo(LPCWSTR pwzUrl, QUERYOPTION QueryOption, - DWORD dwQueryFlags, LPVOID pvBuffer, DWORD cbBuffer, DWORD * pcbBuffer, - DWORD dwReserved) -{ - FIXME("(%s, %x, %x, %p, %x, %p, %x): stub\n", debugstr_w(pwzUrl), - QueryOption, dwQueryFlags, pvBuffer, cbBuffer, pcbBuffer, dwReserved); - return S_OK; -} - -/*********************************************************************** * IsAsyncMoniker (URLMON.@) */ HRESULT WINAPI IsAsyncMoniker(IMoniker *pmk) @@ -1193,6 +1168,8 @@ { LPBC pbc = NULL; HRESULT hr = E_INVALIDARG; + + TRACE("(%p %08x %p %s %p)\n", pmk, grfOpt, pbsc, debugstr_guid(iidResult), ppvResult); if (pmk && ppvResult) { @@ -1437,12 +1414,12 @@ if(szURL) { len = MultiByteToWideChar(CP_ACP, 0, szURL, -1, NULL, 0); - url = urlmon_alloc(len*sizeof(WCHAR)); + url = heap_alloc(len*sizeof(WCHAR)); MultiByteToWideChar(CP_ACP, 0, szURL, -1, url, -1); } if(szFileName) - file_name = urlmon_alloc(dwBufLength*sizeof(WCHAR)); + file_name = heap_alloc(dwBufLength*sizeof(WCHAR)); hres = URLDownloadToCacheFileW(lpUnkCaller, url, file_name, dwBufLength*sizeof(WCHAR), dwReserved, pBSC); @@ -1450,8 +1427,8 @@ if(SUCCEEDED(hres) && file_name) WideCharToMultiByte(CP_ACP, 0, file_name, -1, szFileName, dwBufLength, NULL, NULL); - urlmon_free(url); - urlmon_free(file_name); + heap_free(url); + heap_free(file_name); return hres; } Modified: trunk/reactos/dll/win32/urlmon/umstream.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/umstream.… ============================================================================== --- trunk/reactos/dll/win32/urlmon/umstream.c (original) +++ trunk/reactos/dll/win32/urlmon/umstream.c Mon Jan 14 19:39:00 2008 @@ -19,23 +19,14 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <stdarg.h> - -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" +#include "urlmon_main.h" + #include "winreg.h" #include "winternl.h" -#include "winuser.h" -#include "objbase.h" -#include "wine/debug.h" -#include "wine/unicode.h" -#include "ole2.h" -#include "urlmon.h" #include "wininet.h" #include "shlwapi.h" -#include "urlmon_main.h" + +#include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); @@ -54,7 +45,7 @@ HRESULT hr; size = (strlenW(pszURL)+1)*sizeof(WCHAR); - url = urlmon_alloc(size); + url = heap_alloc(size); memcpy(url, pszURL, size); for (c = url; *c && *c != '#' && *c != '?'; ++c) @@ -72,7 +63,7 @@ else hr = 0; - urlmon_free(url); + heap_free(url); if (hr) return hr; @@ -99,13 +90,13 @@ } } - ucstr = urlmon_alloc_zero(sizeof(IUMCacheStream)); + ucstr = heap_alloc_zero(sizeof(IUMCacheStream)); if(ucstr) { - ucstr->pszURL = urlmon_alloc_zero(sizeof(WCHAR) * (lstrlenW(pszURL) + 1)); + ucstr->pszURL = heap_alloc_zero(sizeof(WCHAR) * (lstrlenW(pszURL) + 1)); if (ucstr->pszURL) { - ucstr->pszFileName = urlmon_alloc_zero(sizeof(WCHAR) * (lstrlenW(pszFileName) + 1)); + ucstr->pszFileName = heap_alloc_zero(sizeof(WCHAR) * (lstrlenW(pszFileName) + 1)); if (ucstr->pszFileName) { ucstr->lpVtbl=&stvt; @@ -119,9 +110,9 @@ return S_OK; } - urlmon_free(ucstr->pszURL); + heap_free(ucstr->pszURL); } - urlmon_free(ucstr); + heap_free(ucstr); } CloseHandle(handle); if (phfile) @@ -207,9 +198,9 @@ TRACE(" destroying UMCacheStream (%p)\n",This); UMCloseCacheFileStream(This); CloseHandle(This->handle); - urlmon_free(This->pszFileName); - urlmon_free(This->pszURL); - urlmon_free(This); + heap_free(This->pszFileName); + heap_free(This->pszURL); + heap_free(This); } return refCount; } @@ -591,7 +582,7 @@ return E_INVALIDARG; len = MultiByteToWideChar(CP_ACP, 0, szURL, -1, NULL, 0); - szURLW = urlmon_alloc(len * sizeof(WCHAR)); + szURLW = heap_alloc(len * sizeof(WCHAR)); if (!szURLW) { *ppStream = NULL; @@ -601,7 +592,7 @@ hr = URLOpenBlockingStreamW(pCaller, szURLW, ppStream, dwReserved, lpfnCB); - urlmon_free(szURLW); + heap_free(szURLW); return hr; } @@ -643,14 +634,14 @@ return E_INVALIDARG; len = MultiByteToWideChar(CP_ACP, 0, szURL, -1, NULL, 0); - szURLW = urlmon_alloc(len * sizeof(WCHAR)); + szURLW = heap_alloc(len * sizeof(WCHAR)); if (!szURLW) return E_OUTOFMEMORY; MultiByteToWideChar(CP_ACP, 0, szURL, -1, szURLW, len); hr = URLOpenStreamW(pCaller, szURLW, dwReserved, lpfnCB); - urlmon_free(szURLW); + heap_free(szURLW); return hr; } Modified: trunk/reactos/dll/win32/urlmon/urlmon.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/urlmon.rb… ============================================================================== --- trunk/reactos/dll/win32/urlmon/urlmon.rbuild (original) +++ trunk/reactos/dll/win32/urlmon/urlmon.rbuild Mon Jan 14 19:39:00 2008 @@ -1,5 +1,6 @@ <?xml version="1.0"?> <!DOCTYPE module SYSTEM "../../../tools/rbuild/project.dtd"> +<group> <module name="urlmon" type="win32dll" baseaddress="${BASEADDRESS_URLMON}" installbase="system32" installname="urlmon.dll" allowwarnings="true"> <autoregister infsection="OleControlDlls" type="Both" /> <importlibrary definition="urlmon.spec.def" /> @@ -35,3 +36,4 @@ <file>rsrc.rc</file> <file>urlmon.spec</file> </module> +</group> Modified: trunk/reactos/dll/win32/urlmon/urlmon_main.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/urlmon_ma… ============================================================================== --- trunk/reactos/dll/win32/urlmon/urlmon_main.c (original) +++ trunk/reactos/dll/win32/urlmon/urlmon_main.c Mon Jan 14 19:39:00 2008 @@ -20,21 +20,15 @@ #include <stdarg.h> -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" +#include "urlmon_main.h" + #include "winreg.h" #define NO_SHLWAPI_REG #include "shlwapi.h" #include "wine/debug.h" -#include "wine/unicode.h" - -#include "winuser.h" + #include "urlmon.h" -#include "urlmon_main.h" -#include "ole2.h" WINE_DEFAULT_DEBUG_CHANNEL(urlmon); Modified: trunk/reactos/dll/win32/urlmon/urlmon_main.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/urlmon/urlmon_ma… ============================================================================== --- trunk/reactos/dll/win32/urlmon/urlmon_main.h (original) +++ trunk/reactos/dll/win32/urlmon/urlmon_main.h Mon Jan 14 19:39:00 2008 @@ -21,8 +21,17 @@ #include <stdarg.h> +#define COBJMACROS +#define NONAMELESSUNION +#define NONAMELESSSTRUCT + #include "windef.h" #include "winbase.h" +#include "winuser.h" +#include "ole2.h" +#include "urlmon.h" + +#include "wine/unicode.h" extern HINSTANCE URLMON_hInstance; extern HRESULT SecManagerImpl_Construct(IUnknown *pUnkOuter, LPVOID *ppobj); @@ -59,28 +68,45 @@ IInternetProtocolInfo *get_protocol_info(LPCWSTR url); HRESULT get_protocol_handler(LPCWSTR url, CLSID *clsid, IClassFactory **ret); -HRESULT start_binding(LPCWSTR url, IBindCtx *pbc, REFIID riid, void **ppv); +HRESULT bind_to_storage(LPCWSTR url, IBindCtx *pbc, REFIID riid, void **ppv); +HRESULT bind_to_object(IMoniker *mon, LPCWSTR url, IBindCtx *pbc, REFIID riid, void **ppv); -HRESULT create_binding_protocol(LPCWSTR url, IInternetProtocol **protocol); +HRESULT create_binding_protocol(LPCWSTR url, BOOL from_urlmon, IInternetProtocol **protocol); +void set_binding_sink(IInternetProtocol *bind_protocol, IInternetProtocolSink *sink); -static inline void *urlmon_alloc(size_t len) +static inline void *heap_alloc(size_t len) { return HeapAlloc(GetProcessHeap(), 0, len); } -static inline void *urlmon_alloc_zero(size_t len) +static inline void *heap_alloc_zero(size_t len) { return HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, len); } -static inline void *urlmon_realloc(void *mem, size_t len) +static inline void *heap_realloc(void *mem, size_t len) { return HeapReAlloc(GetProcessHeap(), 0, mem, len); } -static inline BOOL urlmon_free(void *mem) +static inline BOOL heap_free(void *mem) { return HeapFree(GetProcessHeap(), 0, mem); } +static inline LPWSTR heap_strdupW(LPCWSTR str) +{ + LPWSTR ret = NULL; + + if(str) { + DWORD size; + + size = (strlenW(str)+1)*sizeof(WCHAR); + ret = heap_alloc(size); + memcpy(ret, str, size); + } + + return ret; +} + #endif /* __WINE_URLMON_MAIN_H */

1 0

[hpoussin] 31795: Add missing constants/prototypes
by hpoussin＠svn.reactos.org 14 Jan '08

14 Jan '08

Author: hpoussin Date: Mon Jan 14 19:38:35 2008 New Revision: 31795 URL: http://svn.reactos.org/svn/reactos?rev=31795&view=rev Log: Add missing constants/prototypes Modified: trunk/reactos/include/psdk/urlmon.idl Modified: trunk/reactos/include/psdk/urlmon.idl URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/urlmon.idl?re… ============================================================================== --- trunk/reactos/include/psdk/urlmon.idl (original) +++ trunk/reactos/include/psdk/urlmon.idl Mon Jan 14 19:38:35 2008 @@ -546,8 +546,11 @@ typedef [unique] IMonikerProp *LPMONIKERPROP; typedef enum { - MIMETYPEPROP = 0x00000000, - USE_SRC_URL = 0x00000001 + MIMETYPEPROP = 0, + USE_SRC_URL = 1, + CLASSIDPROP = 2, + TRUSTEDDOWNLOADPROP = 3, + POPUPLEVELPROP = 4 } MONIKERPROPERTY; HRESULT PutProperty( @@ -1264,6 +1267,7 @@ cpp_quote("HRESULT WINAPI GetSoftwareUpdateInfo( LPCWSTR szDistUnit, LPSOFTDISTINFO psdi);") cpp_quote("HRESULT WINAPI FaultInIEFeature(HWND,uCLSSPEC*,QUERYCONTEXT*,DWORD);") cpp_quote("HRESULT WINAPI FindMimeFromData(LPBC,LPCWSTR,LPVOID,DWORD,LPCWSTR,DWORD,LPWSTR*,DWORD);") +cpp_quote("HRESULT WINAPI GetClassFileOrMime(LPBC,LPCWSTR,LPVOID,DWORD,LPCWSTR,DWORD,CLSID*);") cpp_quote("HRESULT WINAPI HlinkGoBack(IUnknown*);") cpp_quote("HRESULT WINAPI HlinkGoForward(IUnknown*);") cpp_quote("HRESULT WINAPI HlinkNavigateMoniker(IUnknown*,IMoniker*);")

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs