Project

General

Profile

Actions

Bug #5859

closed

Wt 4 : Possibly unitialized value

Added by Hyeoni Hwasoo over 6 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
Start date:
08/07/2017
Due date:
% Done:

0%

Estimated time:

Description

Hi. Valgrind found that :

==5751== Thread 7:
==5751== Conditional jump or move depends on uninitialised value(s)
==5751==    at 0x5FED29A: Wt::WTable::createRowDomElement(int, bool, Wt::WApplication*) (WTable.C:313)
==5751==    by 0x5FEED3E: Wt::WTable::getDomChanges(std::vector<Wt::DomElement*, std::allocator<Wt::DomElement*> >&, Wt::WApplication*) (WTable.C:363)
==5751==    by 0x606286C: Wt::WWebWidget::getSDomChanges(std::vector<Wt::DomElement*, std::allocator<Wt::DomElement*> >&, Wt::WApplication*) (WWebWidget.C:2082)
==5751==    by 0x62307E1: Wt::WebRenderer::collectChanges(std::vector<Wt::DomElement*, std::allocator<Wt::DomElement*> >&) (WebRenderer.C:1656)
==5751==    by 0x6230941: Wt::WebRenderer::collectJS(Wt::WStringStream*) (WebRenderer.C:1767)
==5751==    by 0x62321A1: Wt::WebRenderer::collectJavaScriptUpdate(Wt::WStringStream&) (WebRenderer.C:1690)
==5751==    by 0x62328F1: Wt::WebRenderer::collectJavaScript() (WebRenderer.C:856)
==5751==    by 0x6232D8F: Wt::WebRenderer::serveJavaScriptUpdate(Wt::WebResponse&) (WebRenderer.C:614)
==5751==    by 0x62215EE: Wt::WebSession::notify(Wt::WEvent const&) (WebSession.C:2185)
==5751==    by 0x6224E02: Wt::WebSession::pushUpdates() (WebSession.C:2062)
==5751==    by 0x622B8A1: Wt::WebSocketMessage::flush(Wt::WebRequest::ResponseState, std::function<void (Wt::WebWriteEvent)> const&) (WebSocketMessage.C:29)
==5751==    by 0x621539D: Wt::WebSession::Handler::flushResponse() (WebSession.C:1074)

So, I propose the attached patch.


Files

Actions #1

Updated by Roel Standaert over 6 years ago

  • Status changed from New to Implemented @Emweb

I fixed it. Thanks for reporting this issue.

Actions #2

Updated by Roel Standaert over 6 years ago

  • Status changed from Implemented @Emweb to Resolved
Actions #3

Updated by Roel Standaert over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF