Small Memory Leak in selenium/cpp/iedriver/Script.cpp

[somanuell]

At line 194, the ITMLDocument2::geHt_designMode COM method is called.
That method takes an unique [out][retval] BSTR *p parameter, but the argument provided is a NOT an empty CComBSTR. The actual BSTR stored in the smart pointer is leaked.

Suggested FIX:

CComBSTR design_mode;
this->script_engine_host_->get_designMode(&design_mode);
if ( design_mode ) {
  design_mode.ToLower();
  if (design_mode == L"on") {
    CComBSTR set_design_mode = L"off";
    this->script_engine_host_->put_designMode(set_design_mode);
  }
}

OS: Windows 10
Browser: IE11

[github-actions]

This issue is stale because it has been open 365 days with no activity. Remove stale label or comment or this will be closed in 14 days.

[github-actions]

This issue was closed because it has been stalled for 14 days with no activity.

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.