Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • World
  • Users
  • Groups
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Code Project
  1. Home
  2. General Programming
  3. C / C++ / MFC
  4. ShellExecute() launch .CHM and PDF file not work in Window 7

ShellExecute() launch .CHM and PDF file not work in Window 7

Scheduled Pinned Locked Moved C / C++ / MFC
helpquestionc++
25 Posts 6 Posters 0 Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • A Andraw111

    Now we have another small project also created in C++, it also launch .CHM file from help menu: void CMainFrame::LaunchHelp(CString FilePath) { SHELLEXECUTEINFO sei; sei.cbSize = sizeof(SHELLEXECUTEINFO); sei.fMask = NULL; sei.hwnd = NULL; sei.lpVerb = "open"; sei.lpFile = FilePath; sei.lpParameters= NULL; sei.nShow = SW_SHOWMAXIMIZED; sei.hInstApp = NULL; sei.lpIDList = NULL; sei.lpClass = NULL; sei.hkeyClass = NULL; sei.dwHotKey = NULL; sei.hIcon = NULL; sei.hProcess = NULL; sei.lpDirectory = NULL; int ReturnCode = ::ShellExecuteEx(&sei); switch(ReturnCode) { case ERROR_FILE_NOT_FOUND: AfxMessageBox("The specified file was not found."); break; case ERROR_PATH_NOT_FOUND: AfxMessageBox("The specified path was not found."); break; case ERROR_ACCESS_DENIED: AfxMessageBox("Access to the specified file is denied."); break; } } I works fine in Window 7, I copy this function to my application (also created in C++), but after compile and test it in Window 7, still doesn't work, do we need change some settings to make app work in Window 7? I compare the settings, the only difference are that 1: in C++ - >Code Generation -> Use run time library: I use Multithreaded DLL, they use Multlhreaded; 2: in General Microsoft Foundation Classes: I use Use MFC in a Shared Library; they use Use MFC in a Static Library. Thanks!

    S Offline
    S Offline
    Software_Developer
    wrote on last edited by
    #14

    The code below works and launches a helpfile in Windws 7.

    #include
    #include
    #include

    int main()
    {

      SHELLEXECUTEINFO sei = {0};
      sei.cbSize = sizeof(SHELLEXECUTEINFO);
      sei.lpVerb = \_T("open");
      sei.lpFile = \_T("C:\\\\windows\\\\help.CHM");
      sei.fMask = SEE\_MASK\_INVOKEIDLIST;
      sei.nShow = SW\_SHOWDEFAULT;
    
      DWORD exitCode;
      int RetVal = GetExitCodeProcess(sei.hProcess, &exitCode);
      cout<<"Errorcode "<
    
    A 2 Replies Last reply
    0
    • A Andraw111

      Dear all, HINSTANCE handle = ShellExecute(NULL, NULL, strHlpFile, NULL, NULL, SW_SHOW); I create a application in MFC, it works fine in Window XP and return 42 value, but if run it in Window 7, it cannot open the .CHM or PDF file, the return code is 8. What's the problem? how can I solve issue? I try to use system() to launch .CHM or PDF file, it works fine in both Window XP and Window 7, but it has MS-DOS black screen there, how can I let MS-DOS not shown? Thank you very much!

      C Offline
      C Offline
      Charles Oppermann
      wrote on last edited by
      #15

      Have you initialized COM first? From the remarks section on ShellExecute()[^]:

      CoInitializeEx(NULL, COINIT_APARTMENTTHREADED | COINIT_DISABLE_OLE1DDE)

      Can you use ShellExecuteEx[^]? ShellExecuteEx is preferable in this case because you'll get better result information.

      Andraw111 wrote:

      I try to use system() to launch .CHM or PDF file, it works fine in both Window XP and Window 7, but it has MS-DOS black screen there, how can I let MS-DOS not shown?

      The system() C runtime function would not be appropriate here, as you are a Win32 application. Try this, from a command prompt, type start myfile.chm and report on what happens. Same for Runas.

      /* Charles Oppermann */ http://weblogs.asp.net/chuckop

      1 Reply Last reply
      0
      • A Andraw111

        Now we have another small project also created in C++, it also launch .CHM file from help menu: void CMainFrame::LaunchHelp(CString FilePath) { SHELLEXECUTEINFO sei; sei.cbSize = sizeof(SHELLEXECUTEINFO); sei.fMask = NULL; sei.hwnd = NULL; sei.lpVerb = "open"; sei.lpFile = FilePath; sei.lpParameters= NULL; sei.nShow = SW_SHOWMAXIMIZED; sei.hInstApp = NULL; sei.lpIDList = NULL; sei.lpClass = NULL; sei.hkeyClass = NULL; sei.dwHotKey = NULL; sei.hIcon = NULL; sei.hProcess = NULL; sei.lpDirectory = NULL; int ReturnCode = ::ShellExecuteEx(&sei); switch(ReturnCode) { case ERROR_FILE_NOT_FOUND: AfxMessageBox("The specified file was not found."); break; case ERROR_PATH_NOT_FOUND: AfxMessageBox("The specified path was not found."); break; case ERROR_ACCESS_DENIED: AfxMessageBox("Access to the specified file is denied."); break; } } I works fine in Window 7, I copy this function to my application (also created in C++), but after compile and test it in Window 7, still doesn't work, do we need change some settings to make app work in Window 7? I compare the settings, the only difference are that 1: in C++ - >Code Generation -> Use run time library: I use Multithreaded DLL, they use Multlhreaded; 2: in General Microsoft Foundation Classes: I use Use MFC in a Shared Library; they use Use MFC in a Static Library. Thanks!

        C Offline
        C Offline
        Charles Oppermann
        wrote on last edited by
        #16

        You're calling ShellExecute_**Ex**_ but testing the return value for the for SE_ERR_ values? With ShellExecuteEx[^], the return value is true/false and the more information is provided by GetLastError(). For backward compatibility, the SE_ERR_ codes are placed in hInst member of the structure. Try this instead:

        SHELLEXECUTEINFO sei = { sizeof(sei) }; // Will zero out the structure, no need to set NULL on everything
        sei.fMask = SEE_MASK_NOASYNC // Important when you don't have a message loop
        sei.lpFile = FilePath;
        sei.nShow = SW_SHOWMAXIMIZED; // Annoying to force apps to open full screen
        if (false == ShellExecuteEx(&sei))
        {
        // Check GetLastError() here
        }

        /* Charles Oppermann */ http://weblogs.asp.net/chuckop

        S A 2 Replies Last reply
        0
        • C Charles Oppermann

          You're calling ShellExecute_**Ex**_ but testing the return value for the for SE_ERR_ values? With ShellExecuteEx[^], the return value is true/false and the more information is provided by GetLastError(). For backward compatibility, the SE_ERR_ codes are placed in hInst member of the structure. Try this instead:

          SHELLEXECUTEINFO sei = { sizeof(sei) }; // Will zero out the structure, no need to set NULL on everything
          sei.fMask = SEE_MASK_NOASYNC // Important when you don't have a message loop
          sei.lpFile = FilePath;
          sei.nShow = SW_SHOWMAXIMIZED; // Annoying to force apps to open full screen
          if (false == ShellExecuteEx(&sei))
          {
          // Check GetLastError() here
          }

          /* Charles Oppermann */ http://weblogs.asp.net/chuckop

          S Offline
          S Offline
          Software_Developer
          wrote on last edited by
          #17

          Right, GetExitCodeProcess() likewise also gets the error code.

          #include
          #include
          #include

          int main()
          {

            SHELLEXECUTEINFO sei = {0};
            sei.cbSize = sizeof(SHELLEXECUTEINFO);
            sei.lpVerb = \_T("open");
            sei.lpFile = \_T("C:\\\\windows\\\\help.CHM");
            sei.fMask = SEE\_MASK\_INVOKEIDLIST;
            sei.nShow = SW\_SHOWDEFAULT;
          
            DWORD exitCode;
            int RetVal = GetExitCodeProcess(sei.hProcess, &exitCode);
            cout<<"Errorcode "<
          
          1 Reply Last reply
          0
          • S Software_Developer

            The code below works and launches a helpfile in Windws 7.

            #include
            #include
            #include

            int main()
            {

              SHELLEXECUTEINFO sei = {0};
              sei.cbSize = sizeof(SHELLEXECUTEINFO);
              sei.lpVerb = \_T("open");
              sei.lpFile = \_T("C:\\\\windows\\\\help.CHM");
              sei.fMask = SEE\_MASK\_INVOKEIDLIST;
              sei.nShow = SW\_SHOWDEFAULT;
            
              DWORD exitCode;
              int RetVal = GetExitCodeProcess(sei.hProcess, &exitCode);
              cout<<"Errorcode "<
            
            A Offline
            A Offline
            Andraw111
            wrote on last edited by
            #18

            Now I cannot simply say that ::ShellExecuteEx(&sei) doesn't work in Window 7. The problem is that it doesn't work in my application created before. I have a app created using VC++ 6.0 last year, everything is fine except that I cannot open .CHM file from it. Today I create a new application in MFC in VC++ 6.0, then use the exactly same function as the old application, it can open the .CHM file without any problem. I check all the project settings for both application, they are same, why same function works in one application, but failed in another application?

            1 Reply Last reply
            0
            • C Charles Oppermann

              You're calling ShellExecute_**Ex**_ but testing the return value for the for SE_ERR_ values? With ShellExecuteEx[^], the return value is true/false and the more information is provided by GetLastError(). For backward compatibility, the SE_ERR_ codes are placed in hInst member of the structure. Try this instead:

              SHELLEXECUTEINFO sei = { sizeof(sei) }; // Will zero out the structure, no need to set NULL on everything
              sei.fMask = SEE_MASK_NOASYNC // Important when you don't have a message loop
              sei.lpFile = FilePath;
              sei.nShow = SW_SHOWMAXIMIZED; // Annoying to force apps to open full screen
              if (false == ShellExecuteEx(&sei))
              {
              // Check GetLastError() here
              }

              /* Charles Oppermann */ http://weblogs.asp.net/chuckop

              A Offline
              A Offline
              Andraw111
              wrote on last edited by
              #19

              Yes, you are right, ShellExecuteEx() return true or false. But still cannot solve my problem.

              C 1 Reply Last reply
              0
              • S Software_Developer

                The code below works and launches a helpfile in Windws 7.

                #include
                #include
                #include

                int main()
                {

                  SHELLEXECUTEINFO sei = {0};
                  sei.cbSize = sizeof(SHELLEXECUTEINFO);
                  sei.lpVerb = \_T("open");
                  sei.lpFile = \_T("C:\\\\windows\\\\help.CHM");
                  sei.fMask = SEE\_MASK\_INVOKEIDLIST;
                  sei.nShow = SW\_SHOWDEFAULT;
                
                  DWORD exitCode;
                  int RetVal = GetExitCodeProcess(sei.hProcess, &exitCode);
                  cout<<"Errorcode "<
                
                A Offline
                A Offline
                Andraw111
                wrote on last edited by
                #20

                I check the return code by: int RetVal = GetExitCodeProcess(sei.hProcess, &exitCode); both XP and window 7 return 0.

                1 Reply Last reply
                0
                • A Andraw111

                  Yes, you are right, ShellExecuteEx() return true or false. But still cannot solve my problem.

                  C Offline
                  C Offline
                  Charles Oppermann
                  wrote on last edited by
                  #21

                  Andraw111 wrote:

                  Yes, you are right, ShellExecuteEx() return true or false.
                  But still cannot solve my problem.

                  Okay, but what is it returning? True or False? If false, did you check GetLastError()? What was it's value?

                  /* Charles Oppermann */ http://weblogs.asp.net/chuckop

                  A 2 Replies Last reply
                  0
                  • C Charles Oppermann

                    Andraw111 wrote:

                    Yes, you are right, ShellExecuteEx() return true or false.
                    But still cannot solve my problem.

                    Okay, but what is it returning? True or False? If false, did you check GetLastError()? What was it's value?

                    /* Charles Oppermann */ http://weblogs.asp.net/chuckop

                    A Offline
                    A Offline
                    Andraw111
                    wrote on last edited by
                    #22

                    Thanks for reply, it return 8. But strange thing is that if I create a brand new MFC project and use the same codes to open .CHM file, it works fine, why?

                    C 1 Reply Last reply
                    0
                    • C Charles Oppermann

                      Andraw111 wrote:

                      Yes, you are right, ShellExecuteEx() return true or false.
                      But still cannot solve my problem.

                      Okay, but what is it returning? True or False? If false, did you check GetLastError()? What was it's value?

                      /* Charles Oppermann */ http://weblogs.asp.net/chuckop

                      A Offline
                      A Offline
                      Andraw111
                      wrote on last edited by
                      #23

                      Finally I found what cause the .CHM file cannot open in Window 7. In my MFC application, I have a project setting: Link -> Output ->Reserve field, I put "0x40000000". I create a testing application, if I don't set "0x40000000" to Reserve field, it works fine, otherwise, it cannot open .CHM file in Window 7. Even I know the reason, I still cannot solve the problem, since our project is very large, if I remove that setting, when I run my application I get "XX MFC application has encountered a problem and needs to close....".

                      1 Reply Last reply
                      0
                      • A Andraw111

                        Thanks for reply, it return 8. But strange thing is that if I create a brand new MFC project and use the same codes to open .CHM file, it works fine, why?

                        C Offline
                        C Offline
                        Charles Oppermann
                        wrote on last edited by
                        #24

                        I strongly suspect that COM initialization is at issue here[^]. Earlier, I mentioned that COM has to be initialized as follows:

                        CoInitializeEx(NULL, COINIT_APARTMENTTHREADED | COINIT_DISABLE_OLE1DDE)

                        This is why I think you're new project works and the old one doesn't. Is your old project using multithreaded COM? If so, the calls might fail. [Just saw your other post that resolves your issue. I'm posting this for other people who might encounter ShellExecute problems.]

                        /* Charles Oppermann */ http://weblogs.asp.net/chuckop

                        A 1 Reply Last reply
                        0
                        • C Charles Oppermann

                          I strongly suspect that COM initialization is at issue here[^]. Earlier, I mentioned that COM has to be initialized as follows:

                          CoInitializeEx(NULL, COINIT_APARTMENTTHREADED | COINIT_DISABLE_OLE1DDE)

                          This is why I think you're new project works and the old one doesn't. Is your old project using multithreaded COM? If so, the calls might fail. [Just saw your other post that resolves your issue. I'm posting this for other people who might encounter ShellExecute problems.]

                          /* Charles Oppermann */ http://weblogs.asp.net/chuckop

                          A Offline
                          A Offline
                          Andraw111
                          wrote on last edited by
                          #25

                          Ok, I will add them, thanks!

                          1 Reply Last reply
                          0
                          Reply
                          • Reply as topic
                          Log in to reply
                          • Oldest to Newest
                          • Newest to Oldest
                          • Most Votes


                          • Login

                          • Don't have an account? Register

                          • Login or register to search.
                          • First post
                            Last post
                          0
                          • Categories
                          • Recent
                          • Tags
                          • Popular
                          • World
                          • Users
                          • Groups