Uploaded image for project: 'Core ReactOS'
  1. Core ReactOS
  2. CORE-13425

Crash in secur32_winetest:schannel trying to delete uninitialized security context

    XMLWordPrintable

Details

    • June 2017
    • 74,884

    Description

      https://reactos.org/testman/compare.php?ids=53323,53325

      kd> k
      ChildEBP RetAddr  
      0012fd34 0040a8e7 secur32!DeleteSecurityContext+0x86 [c:\ros\reactos\reactos\dll\win32\secur32\wrapper.c @ 460]
      0012fe0c 0040903e secur32_winetest!test_InitializeSecurityContext+0x1b7 [c:\ros\reactos\reactos\modules\rostests\winetests\secur32\schannel.c @ 666]
      0012fe14 004108fb secur32_winetest!func_schannel+0x2e [c:\ros\reactos\reactos\modules\rostests\winetests\secur32\schannel.c @ 1037]
      0012fe30 004107ca secur32_winetest!run_test+0xab [c:\ros\reactos\reactos\sdk\include\reactos\wine\test.h @ 692]
      0012fecc 00411caa secur32_winetest!main+0x1ca [c:\ros\reactos\reactos\sdk\include\reactos\wine\test.h @ 748]
      0012ffb4 00411f78 secur32_winetest!__tmainCRTStartup+0x2ba [c:\ros\reactos\reactos\sdk\lib\crt\startup\crtexe.c @ 311]
      0012ffc0 7c773a44 secur32_winetest!mainCRTStartup+0x28 [c:\ros\reactos\reactos\sdk\lib\crt\startup\crtexe.c @ 196]
      0012fff0 00000000 kernel32!BaseProcessStartup+0x54 [c:\ros\reactos\reactos\dll\win32\kernel32\client\proc.c @ 478]
      kd> ?? status
      int 0n0
      kd> ?? context
      struct _SecHandle
         +0x000 dwLower          : 0xcccccccc
         +0x004 dwUpper          : 0xcccccccc
       
       
      err:(..\dll\win32\schannel\schannel_mbedtls.c:296) schan_imp_handshake: Oops! mbedtls_ssl_handshake returned the following error code: -0x7200...
      kd> ?? pInput
      struct _SecBufferDesc * 0x0012fd5c
         +0x000 ulVersion        : 0
         +0x004 cBuffers         : 1
         +0x008 pBuffers         : 0x0012fd84 _SecBuffer

      The crashing test was introduced in https://source.winehq.org/git/wine.git/commit/c1f055d8dfda4cb4349db319a9e8734022d74895?f=dlls/secur32/tests/schannel.c
      We need the code change that goes with it.

      Attachments

        Issue Links

          Activity

            People

              ThFabba ThFabba
              ThFabba ThFabba
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: