Ros-dev August 2015

ros-dev@reactos.org

13 participants
11 discussions

Re: [ros-dev] [ros-diffs] [bfreisen] 68792: [HHPCOMP] Improve functionality of Windows MSVC build. Based on a WIP patch by Michael Fritscher. See CORE-10019.
by Thomas Faber 23 Aug '15

23 Aug '15

On 2015-08-22 13:16, bfreisen(a)svn.reactos.org wrote: > [HHPCOMP] > Improve functionality of Windows MSVC build. Based on a WIP patch by Michael Fritscher. > See CORE-10019. > +/* if O_BINARY is not defined, the system is probably not expecting any such flag */ > +#ifndef O_BINARY > +#define O_BINARY 0 > +#endif > - chm->fd = creat(filename, 0644); > + chm->fd = open(filename, O_RDWR | O_CREAT | O_TRUNC | O_BINARY, 0644); Hmm... fopen wouldn't have this problem ;)

1 0

Re: [ros-dev] [ros-diffs] [hbelusca] 68754: [NTVDM]: Initialize the PSP' memory control block owner name with the file name (without extension, and up to 8 chars) of the started program.
by Thomas Faber 19 Aug '15

19 Aug '15

On 2015-08-18 14:26, hbelusca(a)svn.reactos.org wrote: > [NTVDM]: Initialize the PSP' memory control block owner name with the file name (without extension, and up to 8 chars) of the started program. Shouldn't that use GetShortPathName to make sure the program can find itself on disk? :p

4 4

Follow-up on Hackfest
by Colin Finck 15 Aug '15

15 Aug '15

Hi all, First of all, let me thank everybody who could make it to Aachen for Hackfest! Thanks also go to all the others on IRC, Twitter, etc. who supported us during that time! It was a pleasure to have you all here and simply a great week! :) Now let's spread the word even more, so that everyone can get an idea of our Hackfest: Hackfest report =============== Aleksey suggested (and volunteered!) to write a report about our Hackfest for the website. He wants someone to help him though. Victor, as you did a pretty great job at live streaming every day, would you help Aleksey here? If there are other volunteers, just raise your voice :) Photos ====== I've created a folder "ReactOS Hackfest 2015" in the "Developers" file section on IServ. You already find all my photos there (among them, first group photos from Saturday). Please share your photos as well, either by uploading through the website or through SFTP (WinSCP is a great tool for this). If you're not in the "Developers" section, but participated in the Hackfest, please tell either Christoph, Pierre or me and we will add you. Cheers, Colin

1 0

Visual Studio Support
by Alberto Vaudagna 14 Aug '15

14 Aug '15

Just wonder why no one has yet fixed the compiling errors you get if you use visual studio. All the errors are not code related but linking problem, so I wonder why no body solve all the compiler errors. Thanks, Alberto Vaudagna

3 15

#ReactOS in OpenExpo'15 (Madrid) and other PR stuff
by victor martinez 14 Aug '15

14 Aug '15

Hi guys!I'm proud to confirm that ReactOS project will be shown in OpenExpo'15 in Madrid, 16th June.OpenExpo is one of the greatest open source events in Spain: http://www.openexpo.esWe'll have the opportunity to give a Talk plus some Hacking lab (wish me good luck!). If anyone wants to come, I've one additional free pass. On the other hand, the open source 2/OS project is pushing forward through some Latam guys:https://plus.google.com/112666045931562995118/posts/6k1Xw2yXB7MSeems they're using ReactOS code in their project according with their tweets:https://twitter.com/2osproject/status/578135968219267072(Translation: We're using code of reactos.org and winehq.org, we have had to interact with networking stack with ROS using JAVA)Using Java??? And they seemed to have some issues with ReactOS code:http://twitter.com/2osproject/status/592291632004882433 Keep rockin'!

6 8

Re: [ros-dev] [ros-diffs] [cwittich] 68673: [MSAFD] don't trace an uninitialized UNICODE_STRING
by Thomas Faber 10 Aug '15

10 Aug '15

On 2015-08-10 19:54, cwittich(a)svn.reactos.org wrote: > @@ -40,8 +40,8 @@ > PLIST_ENTRY Helpers; > INT Status; > > - TRACE("AddressFamily %p, SocketType %p, Protocol %p, Group %u, Flags %lx, TransportName %wZ, HelperDllContext %p, HeplperDllData %p, Events %p\n", > - AddressFamily, SocketType, Protocol, Group, Flags, TransportName, HelperDllContext, HelperDllData, Events); > + TRACE("AddressFamily %p, SocketType %p, Protocol %p, Group %u, Flags %lx, HelperDllContext %p, HeplperDllData %p, Events %p\n", > + AddressFamily, SocketType, Protocol, Group, Flags, HelperDllContext, HelperDllData, Events); So change it to %p? This is tracing all function arguments... seems keeping that scheme would be good. (also you could fix the Heplper typo if you're already at it :p)

1 0

Re: [ros-dev] [ros-diffs] [dreimer] 68672: [BROWSEUI] Address is untranslatable by Adam Stachowicz CORE-9977 #resolve #comment Committed (with a Unicode fix), thx ^^
by Thomas Faber 10 Aug '15

10 Aug '15

On 2015-08-10 18:52, dreimer(a)svn.reactos.org wrote: > --- trunk/reactos/dll/win32/browseui/addressband.cpp [iso-8859-1] (original) > +++ trunk/reactos/dll/win32/browseui/addressband.cpp [iso-8859-1] Mon Aug 10 16:52:40 2015 > @@ -93,7 +93,11 @@ > pdbi->ptActual.y = 22; > } > if (pdbi->dwMask & DBIM_TITLE) > - wcscpy(pdbi->wszTitle, L"Address"); > + { > + if (!LoadStringW(_AtlBaseModule.GetResourceInstance(), IDS_ADDRESSBANDLABEL, pdbi->wszTitle, _countof(pdbi->wszTitle))) > + return E_FAIL; > + } As a general rule, try not to use E_FAIL. HRESULT_FROM_WIN32(GetLastError()) would probably be correct here.

1 0

Audio development
by Andrew Greenwood 09 Aug '15

09 Aug '15

Hi everyone, Some of you probably remember me. I was involved with trying to get audio/sound implemented in ReactOS a few years ago. To be honest, my progress was rather poor, I managed to write a few bits of code but never really had a proper understanding of kernel/OS development or CPU/system architectures. Additionally, whilst I could grasp how the multimedia APIs (MME) in NT4 worked, and how they interacted with the drivers (audio device X opens device Y and writes buffered data to it) for Windows 2000 and later, Kernel Streaming was used which partly built on top of the older implementation but added a load of new stuff. This complicated matters as firstly, KS uses COM (which I still struggle to wrap my head around) in kernel-mode, secondly there were multiple components involved between the application and the hardware (wdmaud.drv -> wdmaud.sys -> ks.sys -> portcls.sys -> driver as far as I remember), and finally I really struggled to find example code and thorough documentation describing how it all worked. Ultimately I gave up with that and intended to move to another platform to develop multimedia applications. Not much happened with that. I remember there were complaints from application developers over Kernel Streaming offering poor latency - instead developers would turn to other technologies such as ASIO or DirectSound. So for recent versions of Windows (Vista onwards?) Microsoft implemented a new, user-mode sound system (WASAPI) which again uses COM but is pretty nicely documented, with code samples. Having done a little reading about this, it renewed my interest. It still uses COM, and KS.SYS is still present (I assume for compatibility?) but as a lot of this is implemented in user-space I suspect it'd be easier to develop/test. Aside from this, over the past few years I've also gained a better understanding of topics I didn't know much about previously and I feel more confident at being able to write lower-level stuff that works properly. I've also developed more of a keen interest in how Windows works. So basically I'm considering implementing the current Windows audio system in ReactOS. I'd imagine we don't need to be too concerned about audio drivers themselves (at least, initially) as vendors tend to offer these and it's not like we need sound immediately post-install! The only thing I really lack now is knowledge of how to implement COM classes - at least, outside of using Visual Studio. Anyone got recommended reading or example code for this? Andrew

3 3

Re: [ros-dev] [ros-diffs] [dreimer] 68645: [shell32] Change Trash to Recycle Bin by Lee Schroeder CORE-8391 #resolve #comment Committed the big one with changes to all RC files, because it's the right way to do that. All th...
by Thomas Faber 09 Aug '15

09 Aug '15

Well... since you asked... Really good reason not to: if something's not broken, don't fix it. Really good reason not to: if everything works fine and the user experience is generally Windows-like, we should NOT try to make things more Windows-like. On the contrary. Different is good, both in terms of project philosophy and legally. Really good reason not to: as another non-native speaker... neither of the two terms seems better than the other for any objective reason. Except that 'trash' is something people actually say :p Keep on hacking! ;) On August 9, 2015 9:56:47 AM CEST, dreimer(a)svn.reactos.org wrote: >Author: dreimer >Date: Sun Aug 9 07:56:47 2015 >New Revision: 68645 > >URL: http://svn.reactos.org/svn/reactos?rev=68645&view=rev >Log: >[shell32] >Change Trash to Recycle Bin by Lee Schroeder >CORE-8391 #resolve #comment Committed the big one with changes to all >RC files, because it's the right way to do that. All these strings are >originated from the english resource and NEVER were translated up to >now, so we keep these strings in sync. Second argument. Trash is not >the right word here (YES, a non native dares to say this ^^) We use the >Windows way here, unless someone gives me a REALLY good reason not to >^^ > >Modified: > trunk/reactos/dll/win32/shell32/lang/ca-ES.rc > trunk/reactos/dll/win32/shell32/lang/da-DK.rc > trunk/reactos/dll/win32/shell32/lang/en-US.rc > trunk/reactos/dll/win32/shell32/lang/fi-FI.rc > trunk/reactos/dll/win32/shell32/lang/he-IL.rc > trunk/reactos/dll/win32/shell32/lang/hu-HU.rc > trunk/reactos/dll/win32/shell32/lang/ko-KR.rc > trunk/reactos/dll/win32/shell32/lang/nl-NL.rc > trunk/reactos/dll/win32/shell32/lang/pt-BR.rc > trunk/reactos/dll/win32/shell32/lang/sl-SI.rc > trunk/reactos/dll/win32/shell32/lang/sv-SE.rc > trunk/reactos/dll/win32/shell32/lang/zh-TW.rc > >Modified: trunk/reactos/dll/win32/shell32/lang/ca-ES.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/ca-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/ca-ES.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/ca-ES.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -661,7 +661,7 @@ > /* special folders */ > IDS_DESKTOP "Desktop" > IDS_MYCOMPUTER "My Computer" >- IDS_RECYCLEBIN_FOLDER_NAME "Trash" >+ IDS_RECYCLEBIN_FOLDER_NAME "Recycle Bin" > IDS_CONTROLPANEL "Control Panel" > IDS_ADMINISTRATIVETOOLS "Administrative Tools" > >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "Are you sure you want to delete '%1'?" >IDS_DELETEMULTIPLE_TEXT "Are you sure you want to delete these %1 >items?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Confirm file overwrite" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/da-DK.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/da-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/da-DK.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/da-DK.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -667,7 +667,7 @@ > /* special folders */ > IDS_DESKTOP "Skrivebord" > IDS_MYCOMPUTER "Min Computer" >- IDS_RECYCLEBIN_FOLDER_NAME "Trash" >+ IDS_RECYCLEBIN_FOLDER_NAME "Recycle Bin" > IDS_CONTROLPANEL "Control Panel" > IDS_ADMINISTRATIVETOOLS "Administrative Tools" > >@@ -695,10 +695,10 @@ > IDS_DELETEITEM_TEXT "Er du sikker pÃ¥ du vil slette '%1'?" >IDS_DELETEMULTIPLE_TEXT "Are you sure you want to delete these %1 >items?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" > IDS_OVERWRITEFILE_TEXT "Overskriv fil %1?" > IDS_OVERWRITEFILE_CAPTION "BekrÃ¦ft overskrivelse af fil" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/en-US.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/en-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/en-US.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/en-US.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -661,7 +661,7 @@ > /* special folders */ > IDS_DESKTOP "Desktop" > IDS_MYCOMPUTER "My Computer" >- IDS_RECYCLEBIN_FOLDER_NAME "Trash" >+ IDS_RECYCLEBIN_FOLDER_NAME "Recycle Bin" > IDS_CONTROLPANEL "Control Panel" > IDS_ADMINISTRATIVETOOLS "Administrative Tools" > >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "Are you sure you want to delete '%1'?" >IDS_DELETEMULTIPLE_TEXT "Are you sure you want to delete these %1 >items?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Confirm file overwrite" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/fi-FI.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/fi-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/fi-FI.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/fi-FI.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -661,7 +661,7 @@ > /* special folders */ > IDS_DESKTOP "TyÃ¶pÃ¶ytÃ¤" > IDS_MYCOMPUTER "Oma Tietokone" >- IDS_RECYCLEBIN_FOLDER_NAME "Trash" >+ IDS_RECYCLEBIN_FOLDER_NAME "Recycle Bin" > IDS_CONTROLPANEL "Control Panel" > IDS_ADMINISTRATIVETOOLS "Administrative Tools" > >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "Haluatko varmasti tuhota '%1':n?" > IDS_DELETEMULTIPLE_TEXT "Haluatko varmasti tuhota nÃ¤mÃ¤ %1?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Vahvista Tiedoston Ylikirjoitus" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/he-IL.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/he-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/he-IL.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/he-IL.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "Are you sure you want to delete '%1'?" >IDS_DELETEMULTIPLE_TEXT "Are you sure you want to delete these %1 >items?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Confirm file overwrite" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/hu-HU.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/hu-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/hu-HU.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/hu-HU.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -661,7 +661,7 @@ > /* special folders */ > IDS_DESKTOP "Asztal" > IDS_MYCOMPUTER "SzÃ¡mÃtÃ³gÃ©p" >- IDS_RECYCLEBIN_FOLDER_NAME "Trash" >+ IDS_RECYCLEBIN_FOLDER_NAME "Recycle Bin" > IDS_CONTROLPANEL "Control Panel" > IDS_ADMINISTRATIVETOOLS "Administrative Tools" > >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "Biztosan tÃ¶rÃ¶lni szeretnÃ©d?: '%1'?" >IDS_DELETEMULTIPLE_TEXT "Biztosan tÃ¶rÃ¶lni szeretnÃ©d ezt a(z) %1 >elemet?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "ÃllomÃ¡ny felÃ¼lÃrÃ¡sa" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/ko-KR.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/ko-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/ko-KR.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/ko-KR.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -661,7 +661,7 @@ > /* special folders */ > IDS_DESKTOP "Desktop" > IDS_MYCOMPUTER "My Computer" >- IDS_RECYCLEBIN_FOLDER_NAME "Trash" >+ IDS_RECYCLEBIN_FOLDER_NAME "Recycle Bin" > IDS_CONTROLPANEL "Control Panel" > IDS_ADMINISTRATIVETOOLS "Administrative Tools" > >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "Are you sure you want to delete '%1'?" >IDS_DELETEMULTIPLE_TEXT "Are you sure you want to delete these %1 >items?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Confirm file overwrite" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/nl-NL.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/nl-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/nl-NL.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/nl-NL.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -661,7 +661,7 @@ > /* special folders */ > IDS_DESKTOP "Desktop" > IDS_MYCOMPUTER "My Computer" >- IDS_RECYCLEBIN_FOLDER_NAME "Trash" >+ IDS_RECYCLEBIN_FOLDER_NAME "Recycle Bin" > IDS_CONTROLPANEL "Control Panel" > IDS_ADMINISTRATIVETOOLS "Administrative Tools" > >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "Are you sure you want to delete '%1'?" >IDS_DELETEMULTIPLE_TEXT "Are you sure you want to delete these %1 >items?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Confirm file overwrite" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/pt-BR.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/pt-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/pt-BR.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/pt-BR.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "VocÃª tem certeza que deseja excluir '%1'?" >IDS_DELETEMULTIPLE_TEXT "VocÃª tem certeza que deseja excluir estes %1 >arquivos?" >IDS_DELETESELECTED_TEXT "VocÃª tem certeza que deseja excluir o(s) >arquivo(s) selecionado(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Confirmar sobreescrever arquivo" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/sl-SI.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/sl-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/sl-SI.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/sl-SI.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -661,7 +661,7 @@ > /* special folders */ > IDS_DESKTOP "Desktop" > IDS_MYCOMPUTER "My Computer" >- IDS_RECYCLEBIN_FOLDER_NAME "Trash" >+ IDS_RECYCLEBIN_FOLDER_NAME "Recycle Bin" > IDS_CONTROLPANEL "Control Panel" > IDS_ADMINISTRATIVETOOLS "Administrative Tools" > >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "Are you sure you want to delete '%1'?" >IDS_DELETEMULTIPLE_TEXT "Are you sure you want to delete these %1 >items?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Confirm file overwrite" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/sv-SE.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/sv-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/sv-SE.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/sv-SE.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -661,7 +661,7 @@ > /* special folders */ > IDS_DESKTOP "Desktop" > IDS_MYCOMPUTER "My Computer" >- IDS_RECYCLEBIN_FOLDER_NAME "Trash" >+ IDS_RECYCLEBIN_FOLDER_NAME "Recycle Bin" > IDS_CONTROLPANEL "Control Panel" > IDS_ADMINISTRATIVETOOLS "Administrative Tools" > >@@ -689,10 +689,10 @@ > IDS_DELETEITEM_TEXT "Are you sure you want to delete '%1'?" >IDS_DELETEMULTIPLE_TEXT "Are you sure you want to delete these %1 >items?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Confirm file overwrite" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" > >Modified: trunk/reactos/dll/win32/shell32/lang/zh-TW.rc >URL: >http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/lang/zh-… >============================================================================== >--- trunk/reactos/dll/win32/shell32/lang/zh-TW.rc [iso-8859-1] >(original) >+++ trunk/reactos/dll/win32/shell32/lang/zh-TW.rc [iso-8859-1] Sun Aug >9 07:56:47 2015 >@@ -697,10 +697,10 @@ > IDS_DELETEITEM_TEXT "Are you sure you want to delete '%1'?" >IDS_DELETEMULTIPLE_TEXT "Are you sure you want to delete these %1 >items?" >IDS_DELETESELECTED_TEXT "Are you sure you want to delete the selected >item(s)?" >- IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Trash?" >- IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Trash?" >- IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Trash?" >- IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Trash. Do you >want to delete it instead?" >+ IDS_TRASHITEM_TEXT "Are you sure that you want to send '%1' to the >Recycle Bin?" >+ IDS_TRASHFOLDER_TEXT "Are you sure that you want to send '%1' and >all its content to the Recycle Bin?" >+ IDS_TRASHMULTIPLE_TEXT "Are you sure that you want to send these >%1 items to the Recycle Bin?" >+ IDS_CANTTRASH_TEXT "The item '%1' can't be sent to Recycle Bin. Do >you want to delete it instead?" >IDS_OVERWRITEFILE_TEXT "This folder already contains a file called >'%1'.\n\nDo you want to replace it?" > IDS_OVERWRITEFILE_CAPTION "Confirm file overwrite" >IDS_OVERWRITEFOLDER_TEXT "This folder already contains a folder named >'%1'.\n\nIf the files in the destination folder have the same names as >files in the\nselected folder they will be replaced. Do you still want >to move or copy\nthe folder?" -- Sent from my Android phone with K-9 Mail. Please excuse my brevity.

3 3

Re: [ros-dev] [ros-diffs] [aandrejevic] 68607: [FASTFAT] Irp->UserBuffer being NULL doesn't indicate any error. It could be that the caller really wants the result stored at address NULL (which can be valid, and is valid by...
by Pierre Schweitzer 07 Aug '15

07 Aug '15

This patch, as it, is really dangerous. MS allows null pointer in its Read/Write routines, but they are wrapped inside a SEH block, to prevent nasty things to happen in case of a null pointer dereference. Here we don't, meaning that a null pointer here will cause major damages. Furthermore, your commit regressed ntdll:file: https://reactos.org/sites/all/modules/reactos/testman/compare.php?ids=40042… Regards, On 08/07/2015 05:30 AM, aandrejevic(a)svn.reactos.org wrote: > Author: aandrejevic > Date: Fri Aug 7 03:30:05 2015 > New Revision: 68607 > > URL: http://svn.reactos.org/svn/reactos?rev=68607&view=rev > Log: > [FASTFAT] > Irp->UserBuffer being NULL doesn't indicate any error. It could be that the > caller really wants the result stored at address NULL (which can be valid, > and is valid by default for programs like NTVDM). > > > Modified: > trunk/reactos/drivers/filesystems/fastfat/rw.c > > Modified: trunk/reactos/drivers/filesystems/fastfat/rw.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… > ============================================================================== > --- trunk/reactos/drivers/filesystems/fastfat/rw.c [iso-8859-1] (original) > +++ trunk/reactos/drivers/filesystems/fastfat/rw.c [iso-8859-1] Fri Aug 7 03:30:05 2015 > @@ -656,7 +656,7 @@ > } > > Buffer = VfatGetUserBuffer(IrpContext->Irp, BooleanFlagOn(IrpContext->Irp->Flags, IRP_PAGING_IO)); > - if (!Buffer) > + if (!Buffer && IrpContext->Irp->MdlAddress) > { > Status = STATUS_INVALID_USER_BUFFER; > goto ByeBye; > @@ -927,7 +927,7 @@ > OldFileSize = Fcb->RFCB.FileSize; > > Buffer = VfatGetUserBuffer(IrpContext->Irp, BooleanFlagOn(IrpContext->Irp->Flags, IRP_PAGING_IO)); > - if (!Buffer) > + if (!Buffer && IrpContext->Irp->MdlAddress) > { > Status = STATUS_INVALID_USER_BUFFER; > goto ByeBye; > > -- Pierre Schweitzer <pierre(a)reactos.org> System & Network Administrator Senior Kernel Developer ReactOS Deutschland e.V.

2 1

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-dev August 2015