February 2007 - Wine-Commits

Mikołaj Zalewski : shell32: shlfileop tests: Avoid buffer overflows for paths.
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: dbef32e4ed168b83496a0c60f434e220914673df URL: http://source.winehq.org/git/wine.git/?a=commit;h=dbef32e4ed168b83496a0c60f… Author: Mikołaj Zalewski <mikolaj(a)zalewski.pl> Date: Sun Feb 4 23:52:39 2007 +0100 shell32: shlfileop tests: Avoid buffer overflows for paths. --- dlls/shell32/tests/shlfileop.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/shell32/tests/shlfileop.c b/dlls/shell32/tests/shlfileop.c index b363c5e..23aaeba 100644 --- a/dlls/shell32/tests/shlfileop.c +++ b/dlls/shell32/tests/shlfileop.c @@ -329,8 +329,8 @@ static void test_delete(void) static void test_rename(void) { SHFILEOPSTRUCTA shfo, shfo2; - CHAR from[MAX_PATH]; - CHAR to[MAX_PATH]; + CHAR from[5*MAX_PATH]; + CHAR to[5*MAX_PATH]; DWORD retval; shfo.hwnd = NULL; @@ -438,8 +438,8 @@ static void test_rename(void) static void test_copy(void) { SHFILEOPSTRUCTA shfo, shfo2; - CHAR from[MAX_PATH]; - CHAR to[MAX_PATH]; + CHAR from[5*MAX_PATH]; + CHAR to[5*MAX_PATH]; FILEOP_FLAGS tmp_flags; DWORD retval; @@ -767,8 +767,8 @@ static void test_copy(void) static void test_move(void) { SHFILEOPSTRUCTA shfo, shfo2; - CHAR from[MAX_PATH]; - CHAR to[MAX_PATH]; + CHAR from[5*MAX_PATH]; + CHAR to[5*MAX_PATH]; DWORD retval; shfo.hwnd = NULL;

1 0

Vitaliy Margolen : winex11drv: Do extra error checking when opening XInput device.
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: 8e52754226035ab739b60ab5567619c9fe83a426 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8e52754226035ab739b60ab55… Author: Vitaliy Margolen <wine-patches(a)kievinfo.com> Date: Sun Feb 4 09:43:39 2007 -0700 winex11drv: Do extra error checking when opening XInput device. --- dlls/winex11.drv/wintab.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/winex11.drv/wintab.c b/dlls/winex11.drv/wintab.c index f56bf08..c6b65a1 100644 --- a/dlls/winex11.drv/wintab.c +++ b/dlls/winex11.drv/wintab.c @@ -367,8 +367,9 @@ void X11DRV_LoadTabletInfo(HWND hwnddefa target = &devices[loop]; cursor = &gSysCursor[cursor_target]; + X11DRV_expect_error(data->display, Tablet_ErrorHandler, NULL); opendevice = pXOpenDevice(data->display,target->id); - if (opendevice) + if (!X11DRV_check_error() && opendevice) { unsigned char map[32]; int i;

1 0

Paul Vriens : wininet/ftp.c: Remove unneeded check.
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: 0cc5a68e2cf4eac4fa45c25f1e2ff7b85af495f3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0cc5a68e2cf4eac4fa45c25f1… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Wed Feb 7 09:07:45 2007 +0100 wininet/ftp.c: Remove unneeded check. --- dlls/wininet/ftp.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/dlls/wininet/ftp.c b/dlls/wininet/ftp.c index 841ae3c..b26350f 100644 --- a/dlls/wininet/ftp.c +++ b/dlls/wininet/ftp.c @@ -262,8 +262,6 @@ BOOL WINAPI FTP_FtpPutFileW(LPWININETFTP return FALSE; } - assert( WH_HFTPSESSION == lpwfs->hdr.htype); - /* Clear any error information */ INTERNET_SetLastError(0); hIC = lpwfs->lpAppInfo;

1 0

Misha Koshelev : msi: Make sure to ignore UI level flags in dialog and files as well.
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: d8b00a071d37bfcd13b4152f3f5106021e9d5f0e URL: http://source.winehq.org/git/wine.git/?a=commit;h=d8b00a071d37bfcd13b4152f3… Author: Misha Koshelev <mk144210(a)bcm.tmc.edu> Date: Tue Feb 6 23:53:24 2007 -0600 msi: Make sure to ignore UI level flags in dialog and files as well. --- dlls/msi/action.c | 2 +- dlls/msi/dialog.c | 2 +- dlls/msi/files.c | 2 +- dlls/msi/msipriv.h | 3 +++ 4 files changed, 6 insertions(+), 3 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index 6c2fdf4..1a8fa5a 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -642,7 +642,7 @@ UINT MSI_InstallPackage( MSIPACKAGE *pac msi_apply_transforms( package ); msi_apply_patches( package ); - if ( (msi_get_property_int(package, szUILevel, 0) & 0xf) >= INSTALLUILEVEL_REDUCED ) + if ( (msi_get_property_int(package, szUILevel, 0) & INSTALLUILEVEL_MASK) >= INSTALLUILEVEL_REDUCED ) { package->script->InWhatSequence |= SEQUENCE_UI; rc = ACTION_ProcessUISequence(package); diff --git a/dlls/msi/dialog.c b/dlls/msi/dialog.c index 86f9e00..421105c 100644 --- a/dlls/msi/dialog.c +++ b/dlls/msi/dialog.c @@ -3542,7 +3542,7 @@ UINT msi_spawn_error_dialog( MSIPACKAGE 'M','S','I','E','r','r','o','r','D','i','a','l','o','g','R','e','s','u','l','t',0 }; - if ( msi_get_property_int(package, szUILevel, 0) == INSTALLUILEVEL_NONE ) + if ( (msi_get_property_int(package, szUILevel, 0) & INSTALLUILEVEL_MASK) == INSTALLUILEVEL_NONE ) return ERROR_SUCCESS; if ( !error_dialog ) diff --git a/dlls/msi/files.c b/dlls/msi/files.c index 9af403f..e370b1e 100644 --- a/dlls/msi/files.c +++ b/dlls/msi/files.c @@ -77,7 +77,7 @@ static UINT msi_change_media( MSIPACKAGE static const WCHAR szUILevel[] = {'U','I','L','e','v','e','l',0}; static const WCHAR error_prop[] = {'E','r','r','o','r','D','i','a','l','o','g',0}; - if ( msi_get_property_int(package, szUILevel, 0) == INSTALLUILEVEL_NONE && !gUIHandlerA ) + if ( (msi_get_property_int(package, szUILevel, 0) & INSTALLUILEVEL_MASK) == INSTALLUILEVEL_NONE && !gUIHandlerA ) return ERROR_SUCCESS; error = generate_error_string( package, 1302, 1, mi->disk_prompt ); diff --git a/dlls/msi/msipriv.h b/dlls/msi/msipriv.h index 815b70c..a89da66 100644 --- a/dlls/msi/msipriv.h +++ b/dlls/msi/msipriv.h @@ -46,6 +46,9 @@ #define MSIWORDCOUNT_ADMINISTRATIVE 0x0004 #define MSIWORDCOUNT_PRIVILEGES 0x0008 +/* Install UI level mask for AND operation to exclude flags */ +#define INSTALLUILEVEL_MASK 0x0007 + #define MSITYPE_IS_BINARY(type) (((type) & ~MSITYPE_NULLABLE) == (MSITYPE_STRING|MSITYPE_VALID)) struct tagMSITABLE;

1 0

Paul Vriens : urlmon: Cast-qual warning fixes.
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: f5035577fb630da8b8714024c10dbde8a247e8ec URL: http://source.winehq.org/git/wine.git/?a=commit;h=f5035577fb630da8b8714024c… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Wed Feb 7 08:06:49 2007 +0100 urlmon: Cast-qual warning fixes. --- dlls/urlmon/umon.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/urlmon/umon.c b/dlls/urlmon/umon.c index d593c81..03289ff 100644 --- a/dlls/urlmon/umon.c +++ b/dlls/urlmon/umon.c @@ -44,7 +44,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(urlmon); /* native urlmon.dll uses this key, too */ -static const WCHAR BSCBHolder[] = { '_','B','S','C','B','_','H','o','l','d','e','r','_',0 }; +static WCHAR BSCBHolder[] = { '_','B','S','C','B','_','H','o','l','d','e','r','_',0 }; /*static BOOL registered_wndclass = FALSE;*/ @@ -541,7 +541,7 @@ static HRESULT URLMonikerImpl_BindToStor *ppvObject = (void *) bind->pstrCache; IStream_AddRef((IStream *) bind->pstrCache); - hres = IBindCtx_GetObjectParam(pbc, (LPOLESTR)BSCBHolder, (IUnknown**)&bind->pbscb); + hres = IBindCtx_GetObjectParam(pbc, BSCBHolder, (IUnknown**)&bind->pbscb); if(SUCCEEDED(hres)) { TRACE("Got IBindStatusCallback...\n"); @@ -1296,16 +1296,16 @@ HRESULT WINAPI RegisterBindStatusCallbac if (pbc == NULL || pbsc == NULL) return E_INVALIDARG; - if (SUCCEEDED(IBindCtx_GetObjectParam(pbc, (LPOLESTR)BSCBHolder, (IUnknown **)&prev))) + if (SUCCEEDED(IBindCtx_GetObjectParam(pbc, BSCBHolder, (IUnknown **)&prev))) { - IBindCtx_RevokeObjectParam(pbc, (LPOLESTR)BSCBHolder); + IBindCtx_RevokeObjectParam(pbc, BSCBHolder); if (ppbscPrevious) *ppbscPrevious = prev; else IBindStatusCallback_Release(prev); } - return IBindCtx_RegisterObjectParam(pbc, (LPOLESTR)BSCBHolder, (IUnknown *)pbsc); + return IBindCtx_RegisterObjectParam(pbc, BSCBHolder, (IUnknown *)pbsc); } /*********************************************************************** @@ -1333,11 +1333,11 @@ HRESULT WINAPI RevokeBindStatusCallback( if (pbc == NULL || pbsc == NULL) return E_INVALIDARG; - if (SUCCEEDED(IBindCtx_GetObjectParam(pbc, (LPOLESTR)BSCBHolder, (IUnknown **)&callback))) + if (SUCCEEDED(IBindCtx_GetObjectParam(pbc, BSCBHolder, (IUnknown **)&callback))) { if (callback == pbsc) { - IBindCtx_RevokeObjectParam(pbc, (LPOLESTR)BSCBHolder); + IBindCtx_RevokeObjectParam(pbc, BSCBHolder); hr = S_OK; } IBindStatusCallback_Release(pbsc);

1 0

Paul Vriens : urlmon: Cast-qual warning fix.
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: bb6e47af6cc6f7e9641621f8af228a7b74b2f672 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bb6e47af6cc6f7e9641621f8a… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Wed Feb 7 08:06:41 2007 +0100 urlmon: Cast-qual warning fix. --- dlls/urlmon/umon.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/urlmon/umon.c b/dlls/urlmon/umon.c index 49260ab..d593c81 100644 --- a/dlls/urlmon/umon.c +++ b/dlls/urlmon/umon.c @@ -1605,7 +1605,7 @@ HRESULT WINAPI URLDownloadToCacheFileW(L HRESULT hr; LPWSTR ext; - static const WCHAR header[] = { + static WCHAR header[] = { 'H','T','T','P','/','1','.','0',' ','2','0','0',' ', 'O','K','\\','r','\\','n','\\','r','\\','n',0 }; @@ -1631,7 +1631,7 @@ HRESULT WINAPI URLDownloadToCacheFileW(L modified.dwLowDateTime = 0; if (!CommitUrlCacheEntryW(szURL, cache_path, expire, modified, NORMAL_CACHE_ENTRY, - (LPWSTR)header, sizeof(header), NULL, NULL)) + header, sizeof(header), NULL, NULL)) return E_FAIL; if (lstrlenW(cache_path) > dwBufLength)

1 0

Andrew Talbot : shell32: Declare some functions static.
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: 0b1fb44aaf4173636fc3f31c658653e5ce45c3ce URL: http://source.winehq.org/git/wine.git/?a=commit;h=0b1fb44aaf4173636fc3f31c6… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Tue Feb 6 22:06:26 2007 +0000 shell32: Declare some functions static. --- dlls/shell32/trash.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/shell32/trash.c b/dlls/shell32/trash.c index ccd347f..8a6de4c 100644 --- a/dlls/shell32/trash.c +++ b/dlls/shell32/trash.c @@ -254,7 +254,7 @@ static char *create_trashinfo(const char return NULL; } -void remove_trashinfo_file(const char *info_dir, const char *base_name) +static void remove_trashinfo_file(const char *info_dir, const char *base_name) { char *filename_buffer; @@ -386,7 +386,7 @@ void TRASH_DisposeElement(TRASH_ELEMENT SHFree(element->filename); } -HRESULT TRASH_GetDetails(const TRASH_ELEMENT *element, WIN32_FIND_DATAW *data) +static HRESULT TRASH_GetDetails(const TRASH_ELEMENT *element, WIN32_FIND_DATAW *data) { LPSTR path = NULL; XDG_PARSED_FILE *parsed = NULL; @@ -480,7 +480,7 @@ failed: return ret; } -INT CALLBACK free_item_callback(void *item, void *lParam) +static INT CALLBACK free_item_callback(void *item, void *lParam) { SHFree(item); return TRUE;

1 0

Andrew Talbot : shell32: Declare some functions static.
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: dfe3d5b88cbb9c6cbaf020d93045817cfc2567f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dfe3d5b88cbb9c6cbaf020d93… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Tue Feb 6 22:05:10 2007 +0000 shell32: Declare some functions static. --- dlls/shell32/shellole.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/shell32/shellole.c b/dlls/shell32/shellole.c index e39a57a..e5d8368 100644 --- a/dlls/shell32/shellole.c +++ b/dlls/shell32/shellole.c @@ -55,7 +55,7 @@ static const WCHAR sShell32[12] = {'S',' * Default ClassFactory types */ typedef HRESULT (CALLBACK *LPFNCREATEINSTANCE)(IUnknown* pUnkOuter, REFIID riid, LPVOID* ppvObject); -IClassFactory * IDefClF_fnConstructor(LPFNCREATEINSTANCE lpfnCI, PLONG pcRefDll, REFIID riidInst); +static IClassFactory * IDefClF_fnConstructor(LPFNCREATEINSTANCE lpfnCI, PLONG pcRefDll, REFIID riidInst); /* this table contains all CLSID's of shell32 objects */ static const struct { @@ -397,7 +397,7 @@ static const IClassFactoryVtbl dclfvt; * IDefClF_fnConstructor */ -IClassFactory * IDefClF_fnConstructor(LPFNCREATEINSTANCE lpfnCI, PLONG pcRefDll, REFIID riidInst) +static IClassFactory * IDefClF_fnConstructor(LPFNCREATEINSTANCE lpfnCI, PLONG pcRefDll, REFIID riidInst) { IDefClFImpl* lpclf;

1 0

Mikołaj Zalewski : comctl32: toolbar: Copy bitmaps with CopyImage so that it works also for DIBs ( based on a patch by Oleg Krylov).
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: dadc4248cf901d4a0bcbfaafacb29202f923ecc4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dadc4248cf901d4a0bcbfaafa… Author: Mikołaj Zalewski <mikolaj(a)zalewski.pl> Date: Tue Feb 6 23:12:46 2007 +0100 comctl32: toolbar: Copy bitmaps with CopyImage so that it works also for DIBs (based on a patch by Oleg Krylov). --- dlls/comctl32/toolbar.c | 82 ++++++---------------------------------------- 1 files changed, 11 insertions(+), 71 deletions(-) diff --git a/dlls/comctl32/toolbar.c b/dlls/comctl32/toolbar.c index 2dca16e..a60bf09 100644 --- a/dlls/comctl32/toolbar.c +++ b/dlls/comctl32/toolbar.c @@ -2664,33 +2664,7 @@ TOOLBAR_AddBitmapToImageList(TOOLBAR_INF TRACE("adding hInst=%p nID=%d nButtons=%d\n", bitmap->hInst, bitmap->nID, bitmap->nButtons); /* Add bitmaps to the default image list */ if (bitmap->hInst == NULL) /* a handle was passed */ - { - BITMAP bmp; - HBITMAP hOldBitmapBitmap, hOldBitmapLoad; - HDC hdcImage, hdcBitmap; - - /* copy the bitmap before adding it so that the user's bitmap - * doesn't get modified. - */ - GetObjectW ((HBITMAP)bitmap->nID, sizeof(BITMAP), (LPVOID)&bmp); - - hdcImage = CreateCompatibleDC(0); - hdcBitmap = CreateCompatibleDC(0); - - /* create new bitmap */ - hbmLoad = CreateBitmap (bmp.bmWidth, bmp.bmHeight, bmp.bmPlanes, bmp.bmBitsPixel, NULL); - hOldBitmapBitmap = SelectObject(hdcBitmap, (HBITMAP)bitmap->nID); - hOldBitmapLoad = SelectObject(hdcImage, hbmLoad); - - /* Copy the user's image */ - BitBlt (hdcImage, 0, 0, bmp.bmWidth, bmp.bmHeight, - hdcBitmap, 0, 0, SRCCOPY); - - SelectObject (hdcImage, hOldBitmapLoad); - SelectObject (hdcBitmap, hOldBitmapBitmap); - DeleteDC (hdcImage); - DeleteDC (hdcBitmap); - } + hbmLoad = (HBITMAP)CopyImage((HBITMAP)bitmap->nID, IMAGE_BITMAP, 0, 0, 0); else hbmLoad = CreateMappedBitmap(bitmap->hInst, bitmap->nID, 0, NULL, 0); @@ -4175,7 +4149,6 @@ TOOLBAR_ReplaceBitmap (HWND hwnd, WPARAM TOOLBAR_INFO *infoPtr = TOOLBAR_GetInfoPtr (hwnd); LPTBREPLACEBITMAP lpReplace = (LPTBREPLACEBITMAP) lParam; HBITMAP hBitmap; - HBITMAP hbmLoad = NULL; int i = 0, nOldButtons = 0, pos = 0; int nOldBitmaps, nNewBitmaps = 0; HIMAGELIST himlDef = 0; @@ -4192,16 +4165,6 @@ TOOLBAR_ReplaceBitmap (HWND hwnd, WPARAM else if (lpReplace->hInstOld != 0) FIXME("resources not in the current module not implemented\n"); - if (lpReplace->hInstNew) - { - hbmLoad = LoadBitmapW(lpReplace->hInstNew,(LPWSTR)lpReplace->nIDNew); - hBitmap = hbmLoad; - } - else - { - hBitmap = (HBITMAP) lpReplace->nIDNew; - } - TRACE("To be replaced hInstOld %p nIDOld %x\n", lpReplace->hInstOld, lpReplace->nIDOld); for (i = 0; i < infoPtr->nNumBitmapInfos; i++) { TBITMAP_INFO *tbi = &infoPtr->bitmaps[i]; @@ -4222,11 +4185,17 @@ TOOLBAR_ReplaceBitmap (HWND hwnd, WPARAM if (nOldButtons == 0) { WARN("No hinst/bitmap found! hInst %p nID %x\n", lpReplace->hInstOld, lpReplace->nIDOld); - if (hbmLoad) - DeleteObject (hbmLoad); return FALSE; } + /* copy the bitmap before adding it as ImageList_AddMasked modifies the + * bitmap + */ + if (lpReplace->hInstNew) + hBitmap = LoadBitmapW(lpReplace->hInstNew,(LPWSTR)lpReplace->nIDNew); + else + hBitmap = CopyImage((HBITMAP)lpReplace->nIDNew, IMAGE_BITMAP, 0, 0, 0); + himlDef = GETDEFIMAGELIST(infoPtr, 0); /* fixme: correct? */ nOldBitmaps = ImageList_GetImageCount(himlDef); @@ -4237,35 +4206,9 @@ TOOLBAR_ReplaceBitmap (HWND hwnd, WPARAM if (hBitmap) { - BITMAP bmp; - HBITMAP hOldBitmapBitmap, hOldBitmapLoad, hbmLoad; - HDC hdcImage, hdcBitmap; - - /* copy the bitmap before adding it so that the user's bitmap - * doesn't get modified. - */ - GetObjectW (hBitmap, sizeof(BITMAP), (LPVOID)&bmp); - - hdcImage = CreateCompatibleDC(0); - hdcBitmap = CreateCompatibleDC(0); - - /* create new bitmap */ - hbmLoad = CreateBitmap (bmp.bmWidth, bmp.bmHeight, bmp.bmPlanes, bmp.bmBitsPixel, NULL); - hOldBitmapBitmap = SelectObject(hdcBitmap, hBitmap); - hOldBitmapLoad = SelectObject(hdcImage, hbmLoad); - - /* Copy the user's image */ - BitBlt (hdcImage, 0, 0, bmp.bmWidth, bmp.bmHeight, - hdcBitmap, 0, 0, SRCCOPY); - - SelectObject (hdcImage, hOldBitmapLoad); - SelectObject (hdcBitmap, hOldBitmapBitmap); - DeleteDC (hdcImage); - DeleteDC (hdcBitmap); - - ImageList_AddMasked (himlDef, hbmLoad, comctl32_color.clrBtnFace); + ImageList_AddMasked (himlDef, hBitmap, comctl32_color.clrBtnFace); nNewBitmaps = ImageList_GetImageCount(himlDef); - DeleteObject (hbmLoad); + DeleteObject(hBitmap); } infoPtr->nNumBitmaps = infoPtr->nNumBitmaps - nOldBitmaps + nNewBitmaps; @@ -4274,9 +4217,6 @@ TOOLBAR_ReplaceBitmap (HWND hwnd, WPARAM pos, nOldBitmaps, nNewBitmaps); InvalidateRect(hwnd, NULL, TRUE); - - if (hbmLoad) - DeleteObject (hbmLoad); return TRUE; }

1 0

Andrew Talbot : shell32: Declare some functions static.
by Alexandre Julliard Feb. 7, 2007

Feb. 7, 2007

Module: wine Branch: master Commit: 434c24d83bcd311a953264157f923f2acad4b672 URL: http://source.winehq.org/git/wine.git/?a=commit;h=434c24d83bcd311a953264157… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Tue Feb 6 22:04:31 2007 +0000 shell32: Declare some functions static. --- dlls/shell32/dialogs.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/shell32/dialogs.c b/dlls/shell32/dialogs.c index e5f06c6..eb5c1a1 100644 --- a/dlls/shell32/dialogs.c +++ b/dlls/shell32/dialogs.c @@ -52,8 +52,8 @@ typedef struct typedef BOOL (*LPFNOFN) (OPENFILENAMEA *) ; WINE_DEFAULT_DEBUG_CHANNEL(shell); -INT_PTR CALLBACK RunDlgProc (HWND, UINT, WPARAM, LPARAM) ; -void FillList (HWND, char *) ; +static INT_PTR CALLBACK RunDlgProc (HWND, UINT, WPARAM, LPARAM) ; +static void FillList (HWND, char *) ; /************************************************************************* @@ -116,7 +116,7 @@ void WINAPI RunFileDlg( } /* Dialog procedure for RunFileDlg */ -INT_PTR CALLBACK RunDlgProc (HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) +static INT_PTR CALLBACK RunDlgProc (HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) { int ic ; char *psz, szMsg[256] ; @@ -235,7 +235,7 @@ INT_PTR CALLBACK RunDlgProc (HWND hwnd, } /* This grabs the MRU list from the registry and fills the combo for the "Run" dialog above */ -void FillList (HWND hCb, char *pszLatest) +static void FillList (HWND hCb, char *pszLatest) { HKEY hkey ; /* char szDbgMsg[256] = "" ; */

1 0