diff options
| author | al3x <al3x@b3059339-0415-0410-9bf9-f77b7e298cf2> | 2001-06-11 12:47:48 +0000 |
|---|---|---|
| committer | al3x <al3x@b3059339-0415-0410-9bf9-f77b7e298cf2> | 2001-06-11 12:47:48 +0000 |
| commit | 17a8a234ffabf7357d79e0e794c7ebb101ae2659 (patch) | |
| tree | ac2ccc5ded4b1315d7b3b10ba7ae9bf9bbf7b94a /loader | |
| parent | ee6d3b42d1689b71f027878cc41473de3e6455e1 (diff) | |
| download | mpv-17a8a234ffabf7357d79e0e794c7ebb101ae2659.tar.bz2 mpv-17a8a234ffabf7357d79e0e794c7ebb101ae2659.tar.xz | |
some possible buffer overflow places fixed
git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@1097 b3059339-0415-0410-9bf9-f77b7e298cf2
Diffstat (limited to 'loader')
| -rw-r--r-- | loader/dshow/DS_Filter.c | 4 | ||||
| -rw-r--r-- | loader/vfl.c | 4 | ||||
| -rw-r--r-- | loader/win32.c | 6 |
3 files changed, 7 insertions, 7 deletions
diff --git a/loader/dshow/DS_Filter.c b/loader/dshow/DS_Filter.c index 966b2cde87..d3a530ddb4 100644 --- a/loader/dshow/DS_Filter.c +++ b/loader/dshow/DS_Filter.c @@ -79,14 +79,14 @@ void DS_Filter::Create(char* dllname, const GUID* id, AM_MEDIA_TYPE* in_fmt, AM if (!m_iHandle) { char e[1024]; - sprintf(e, "Could not open DirectShow DLL: %s", dllname); + snprintf(e, 1024, "Could not open DirectShow DLL: %s", dllname); throw FATAL(e); } GETCLASS func=(GETCLASS)GetProcAddress(m_iHandle, "DllGetClassObject"); if (!func) { char e[1024]; - sprintf(e, "Illegal or corrupt DirectShow DLL: %s", dllname); + snprintf(e, 1024, "Illegal or corrupt DirectShow DLL: %s", dllname); throw FATAL(e); } diff --git a/loader/vfl.c b/loader/vfl.c index 19d12c0e44..6af2d42bc1 100644 --- a/loader/vfl.c +++ b/loader/vfl.c @@ -121,7 +121,7 @@ ICOpen(long fccType,long fccHandler,unsigned int wMode) { memcpy(type,&fccType,4);type[4]=0; memcpy(handler,&fccHandler,4);handler[4]=0; - sprintf(codecname,"%s.%s",type,handler); + snprintf(codecname,20,"%s.%s",type,handler); /* Well, lParam2 is in fact a LPVIDEO_OPEN_PARMS, but it has the * same layout as ICOPEN @@ -136,7 +136,7 @@ ICOpen(long fccType,long fccHandler,unsigned int wMode) { /* if (!hdrv) { if (!strcasecmp(type,"vids")) { - sprintf(codecname,"vidc.%s",handler); + snprintf(codecname,20,"vidc.%s",handler); fccType = mmioFOURCC('v','i','d','c'); } // hdrv=OpenDriverA(codecname,"drivers32",(long)&icopen); diff --git a/loader/win32.c b/loader/win32.c index 323c99b1ad..4b06552570 100644 --- a/loader/win32.c +++ b/loader/win32.c @@ -771,7 +771,7 @@ void WINAPI expGetSystemInfo(SYSTEM_INFO* si) } } /* set the CPU type of the current processor */ - sprintf(buf,"CPU %ld",cachedsi.dwProcessorType); + snprintf(buf,20,"CPU %ld",cachedsi.dwProcessorType); continue; } /* old 2.0 method */ @@ -798,7 +798,7 @@ void WINAPI expGetSystemInfo(SYSTEM_INFO* si) } } /* set the CPU type of the current processor */ - sprintf(buf,"CPU %ld",cachedsi.dwProcessorType); + snprintf(buf,20,"CPU %ld",cachedsi.dwProcessorType); continue; } if (!lstrncmpiA(line,"fdiv_bug",strlen("fdiv_bug"))) { @@ -824,7 +824,7 @@ void WINAPI expGetSystemInfo(SYSTEM_INFO* si) /* Create a new processor subkey on a multiprocessor * system */ - sprintf(buf,"%d",x); + snprintf(buf,20,"%d",x); } if (!lstrncmpiA(line,"stepping",strlen("stepping"))) { int x; |