Rack v1: Crash when loading modules in module browser

[ascustomworks]

Rack v1: Crash when loading modules in module browser

[Coirt]

Interesting, yeah confirmed now loading! 🍶

[cschol]

That fixes it. Thanks.

[Coirt]

Seems fixed 👍

[AScustomWorks]

Thanks, can't test on Linux right now, but I think I found the error (missing NULL declaration to avoid the module browser crash):
dd1d381

[cschol]

I'll test it. Thanks.

[Coirt]

There might be a few couple more BPMcalc:
@AScustomWorks

Thread 1 received signal SIGSEGV, Segmentation fault.
0x0000000067eba94e in std::operator<< <char, std::char_traits<char>, std::allocator<char> > (__str=..., __os=...)
    at C:/msys64/mingw64/include/c++/8.2.1/bits/basic_string.h:6323
6323        operator<<(basic_ostream<_CharT, _Traits>& __os,
(gdb) bt
#0  0x0000000067eba94e in std::operator<< <char, std::char_traits<char>, std::allocator<char> > (__str=..., __os=...)
    at C:/msys64/mingw64/include/c++/8.2.1/bits/basic_string.h:6323
#1  TempodisplayWidget::draw (this=0x2e314cd0, args=...)
    at src/BPMCalc.cpp:344
#2  0x00000000004a2388 in rack::widget::Widget::draw (
    this=this@entry=0x2e321860, args=...) at src/widget/Widget.cpp:174
#3  0x000000000048e3cb in rack::app::ModuleWidget::draw (this=0x2e321860,
    args=...) at src/app/ModuleWidget.cpp:259
#4  0x00000000004a2388 in rack::widget::Widget::draw (this=0x2e3217a0,
    args=...) at src/widget/Widget.cpp:174
#5  0x00000000004a2f0f in rack::widget::ZoomWidget::draw (this=0x2e3217a0,
    args=...) at src/widget/ZoomWidget.cpp:38
#6  0x00000000004a2388 in rack::widget::Widget::draw (
    this=this@entry=0x2e3216a0, args=...) at src/widget/Widget.cpp:174
#7  0x00000000004a3a2b in rack::widget::FramebufferWidget::drawFramebuffer (
    this=0x2e3216a0) at src/widget/FramebufferWidget.cpp:187
#8  0x00000000004a32fa in rack::widget::FramebufferWidget::step (
    this=0x2e3216a0) at src/widget/FramebufferWidget.cpp:79
#9  0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d461c0)
    at src/widget/Widget.cpp:151
#10 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d46120)
    at src/widget/Widget.cpp:151
#11 0x00000000004a2bd7 in rack::widget::Widget::step (
    this=this@entry=0x6d3cb60) at src/widget/Widget.cpp:151
#12 0x00000000004a11fb in rack::ui::SequentialLayout::step (this=0x6d3cb60)
    at src/ui/SequentialLayout.cpp:14
#13 0x00000000004a2bd7 in rack::widget::Widget::step (
    this=this@entry=0x6d3ca90) at src/widget/Widget.cpp:151
#14 0x000000000049f0dd in rack::ui::MarginLayout::step (this=0x6d3ca90)
    at src/ui/MarginLayout.cpp:10
#15 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d3c7e0)
    at src/widget/Widget.cpp:151
#16 0x00000000004a2bd7 in rack::widget::Widget::step (
    this=this@entry=0x6d3c750) at src/widget/Widget.cpp:151
#17 0x000000000049ead3 in rack::ui::ScrollWidget::step (this=0x6d3c750)
    at src/ui/ScrollWidget.cpp:36
#18 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d3bd50)
    at src/widget/Widget.cpp:151
#19 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d3bce0)
    at src/widget/Widget.cpp:151
#20 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6cc70d0)
    at src/widget/Widget.cpp:151
#21 0x000000000048e0c4 in rack::app::Scene::step (this=0x6cc70d0)
    at src/app/Scene.cpp:51
#22 0x0000000000454bf0 in rack::Window::run (this=0x6cd3450)
    at src/window.cpp:363
#23 0x000000000079c7dd in main (argc=<optimized out>, argv=<optimized out>)
    at src/main.cpp:175
(gdb) make: *** [Makefile:64: debug] Interrupt

[cschol]

@Coirt Is this in the module browser or when the module is running?

[Coirt]

Browser

[cschol]

Confirmed. It is seemingly timing related whether the crash occurs. A few times I got it to work...but it is definitely an issue. Thanks for reporting.

[cschol]

This should do it:

diff --git a/src/BPMCalc.cpp b/src/BPMCalc.cpp
index f8a353a..30d15f4 100755
--- a/src/BPMCalc.cpp
+++ b/src/BPMCalc.cpp
@@ -312,7 +312,7 @@ struct BPMCalc : Module {
 
 ////////////////////////////////////
 struct TempodisplayWidget : TransparentWidget {
-  std::string *value;
+  std::string *value = NULL;
   std::shared_ptr<Font> font;
 
   TempodisplayWidget() {
@@ -634,4 +634,4 @@ struct BPMCalcWidget : ModuleWidget {
 };
 
 
-Model *modelBPMCalc = createModel<BPMCalc, BPMCalcWidget>("BPMCalc");
\ No newline at end of file
+Model *modelBPMCalc = createModel<BPMCalc, BPMCalcWidget>("BPMCalc");
diff --git a/src/BPMCalc2.cpp b/src/BPMCalc2.cpp
index 62d9d2d..08995a0 100755
--- a/src/BPMCalc2.cpp
+++ b/src/BPMCalc2.cpp
@@ -312,7 +312,7 @@ struct BPMCalc2 : Module {
 
 ////////////////////////////////////
 struct TempodisplayWidget : TransparentWidget {
- std::string *value;
+ std::string *value = NULL;
   std::shared_ptr<Font> font;
 
   TempodisplayWidget() {
@@ -425,4 +425,4 @@ struct BPMCalc2Widget : ModuleWidget {
 };
 
 
-Model *modelBPMCalc2 = createModel<BPMCalc2, BPMCalc2Widget>("BPMCalc2");
\ No newline at end of file
+Model *modelBPMCalc2 = createModel<BPMCalc2, BPMCalc2Widget>("BPMCalc2");

[AScustomWorks]

Oh, missed those two. fixed now:
15e8965
And added "Distortion" tag to the Overdrive FX module in plugin.json
730c764

[cschol]

Thanks!