Important: Please read the Qt Code of Conduct - https://forum.qt.io/topic/113070/qt-code-of-conduct

sometimes qt was ended forcefully, weird.



  • hi,
    i am using qt 5.15.1 and qtcreator 4.13.2, when i press CTRL+R to run the program, sometimes (50% possibility) it was ended forcefully (nothing else is changed by me), but not with the DEBUG mode(F5). i have located it was crashed in the following code snippets,

    QFileInfo file(QCoreApplication::applicationFilePath());
    if (some condition)
    {
    QMessageBox::information(this, "title", QString("%1").arg(file.size()));
    }

    if i run the program in the console, it prints the following things,
    *** Error in './FC': fress(): invalid pointer: 0x............ ***
    you know what, based on this, i modified the code as below,
    QFileInfo file(“~/qt/.....”);
    if (some condition)
    {
    QMessageBox::information(this, "title", QString("%1").arg(file.size()));
    }
    and it will never be crashed here but later somewhether in the same block {}. (trying to contrust an instance of an invalid type, type id:1061)

    anothing weird is, this version is modified from yesterday's version. they are same but the newer version has some more function somewhere after the crash position, but old version never crash.

    it's a big project so it's difficult to post the whole code. pls anybody could give me some advice or opinion?

    thanks,
    Circle



  • @circle said in sometimes qt was ended forcefully, weird.:

    *** Error in './FC': fress(): invalid pointer: 0x............ ***

    So fress() is a method of yours?

    You may be able to use a core dump from not-running-in-debugger to get information about where the crash occurred, e.g. gdb executable core.

    You can Google for qt trying to construct an instance of an invalid type for possibilities about what might be at issue (e.g. possibly a QVariant).



  • free() is not mine, i guess it was the temp QString object by QCoreApplication::applicationFilePath().
    When I debug the program by press F5 (via gdb), it never crash. so do you think core dump with gdb might help? im asking because i am new to qt, linux and gdb.

    in addition, about the invalid type, i googled before but i have no idea how to locate the problem, the code snippet is as below,
    ui->statusbar->addwidget(ui->line1);
    ui->statusbar->addwidget(ui->label1);
    ...



  • @circle
    But your post does not mention free(). I quoted what you wrote:

    *** Error in './FC': fress(): invalid pointer: 0x............ ***

    Please, please don't tell me you chose to type in the message without copy/pasting it, and on top of that got it wrong? You wouldn't do that, would you, because it's not fair on those trying to help you if you just give out wrong information, and for my own part I don't help people if they do that.



  • @circle

    Do you try to delete some object twice?



  • @JonB
    i am sorry, that was what i did. now i connect the local machine to internet so i am going to post more code here,

    MainWindow::MainWindow(QWidget *parent)
        : QMainWindow(parent)
        , ui(new Ui::MainWindow)
    {
        ui->setupUi(this);
        showFullScreen();
    
        ui->dockWidget->setWidget(ui->tableWidget);
    
        ui->line1->hide();
    }
    
    
    void MainWindow::resizeEvent(QResizeEvent *event)
    {
        QMainWindow::resizeEvent(event);
    
        // Your code here.
        static bool bOnlyOnce = true;
        if (bOnlyOnce && isVisible())
        {
            //doing some initialization here
    
            QFileInfo fileInfo(QCoreApplication::applicationFilePath()); 
            if ()
                QMessageBox::information(this, "title", QString("%1").arg(fileInfo.size()));
        
            //doing more staff, basically ui layout work
    
            ui->statusbar->addWidget(ui->lable1, 40); 
        }
    }
    
    

    now the QMessageBox line would crash sometimes(qtcreator says the process was ended forcefully), and if i comment out this line, the ui->statusbar line sometimes would report "Trying to construct an instance of an invalid type, type id: 1061" and the process was ended forcefully.
    i haven't found any rules about "sometimes", sometimes it will crash several times continuously when i press CTRL+R, but sometimes the program works fine.



  • @Pl45m4
    thanks but no. the weird thing is it's working sometimes without changing anything apparently.


  • Lifetime Qt Champion

    Hi,

    Why are you doing that layouting logic that should run only once in your resize event ?



  • @SGaist
    thanks, the gui is supposed to be fullscreen all the time,and some initialization work should be done only once, as well as the layout work which is based on fullscreen.

    anyway, i just found the problem of ui->statusbar line might be related to stylesheet. the original stylesheet set in the property window is as below,
    QStatusBar{border-top: 1px outset grey;}
    QStatusBar::item{border: 0px}
    if i clear this setting, it seems it's not reporting the error(trying to Trying to construct an instance of an invalid type) anymore.
    if i change the stylesheet to some back-ground color, it might be crash again, but not with the error (trying to Trying to construct an instance of an invalid type) anymore.
    even i set the stylesheet in the mainwidow.cpp code, it also might be crashed sometimes.

    actually, i still don't believe it's the root reason, one reason is i don't see anything is wrong with the stylesheet, the other reason it never crash in the old backup version.

    i googled a lot but i haven't found anything helpful.

    i have stuck into here for a long time, based on the information i just mentioned, do you have any further advice?

    thank you all very much for your time and help.


  • Moderators

    @circle are you sure you set bOnlyOnce to false anywhere ? It's not in your code.

    resize events may be queued up already, when its first called, and QmessageBox will spin its own event loop that could be the problem here



  • @J-Hilk
    sorry forget to post that line, the first thing in the if(bOnlyOnce) block is to set bOnlyOnce = false;

    i understand resize events may be queued, and QMessageBox has its own event loop, but sorry i am not very aware of what you said, how it affect?

    from my debug log, it will go to the resize function block after showFullscreen() in the constructor, but it's not visible yet, so after back to the constructor, the system will call resize function again to do the initialization work.

    In addition, when i press F5 to debug the program, i never reproduce the problem. and i can't reproduce the problem with the old backup version, and they are same in the constructor and resize function block.


  • Lifetime Qt Champion

    Can you explain what kind of logic is required to be run in the resize event ?

    Typically I do not see why you would need to add a widget there.



  • @SGaist
    thanks for your time. my app is fullscreen always, and the main reason i do the initialization work in the resize event is, i need to know the size of the fullscreen, as a lot of drawing is based on this size. Most of the drawing is done in the memory, while the small part is done in paintEvent.
    where do you think i should do these things? showEvent?



  • @circle said in sometimes qt was ended forcefully, weird.:

    where do you think i should do these things?

    You can get the size (geometry) of your widget anywhere, without calling the stuff every time you resize.

    If you want to do something just once, consider doing it inside your c'tor or use the showEvent, when you need the actual size.
    (Your widget's size in c'tor and showEvent will / may be different, because of layouts)



  • @Pl45m4 said in sometimes qt was ended forcefully, weird.:

    c'tor

    thanks, i tried to get the real fullscreen's size in the c'tor or in the showEvent, but the size was wrong. that's why i put those work in the resizeEvent and isVisible() block.


  • Lifetime Qt Champion

    @circle said in sometimes qt was ended forcefully, weird.:

    tried to get the real fullscreen's size in the c'tor

    QScreen doesn't work differently if it is called in the ctor or anywhere else.


  • Lifetime Qt Champion

    Depending on what you need, I would rather go with changeEvent and check the QEvent::WindowStateChange event. You will know there when the window is fullscreen.

    But again, doing stuff that depends on the specifics of being fullscreen is a bit surprising. Painting is usually done so that it does not directly depend on one specific size because even if fullscreen you may have different sizes.



  • @Christian-Ehrlicher
    more precisely, what i need is the size of centralwidget when it's fullscreen. so i just use the width() and hegith() to obtain those when it's fullscreen.
    i didn't use QScreen, should i?

    @SGaist
    there may be different ways to get the size, but i still don't have a clue why it produce such an error. it seems have nothing to do with the error, and why sometimes it's working, sometimes not.


  • Lifetime Qt Champion

    @circle said in sometimes qt was ended forcefully, weird.:

    i didn't use QScreen, should i?

    I suggest you to use it so I would say it's worth a try reading the documentation, isn't it?



  • @circle said in sometimes qt was ended forcefully, weird.:

    more precisely, what i need is the size of centralwidget when it's fullscreen.

    Then it doesn't sound like you should be looking at QScreen at all.

    thanks, i tried to get the real fullscreen's size in the c'tor or in the showEvent,

    To get a size of widget (not the full screen, which you shouldn't be interested in) correctly it needs to be shown. If you call it too early, you get rubbish. Constructor is not the place. If in a showEvent(), IIRC you should call the base showEvent() before you try to read the size.



  • @JonB
    even call the base showEvent() first, i still can't read the right size of the centralwidget.
    anyway, the size is not a problem, the problem is that weird errors reported has nothing to do with the code as far as i am concerned.


  • Lifetime Qt Champion

    @circle said in sometimes qt was ended forcefully, weird.:

    i still can't read the right size of the centralwidget.

    Again: use QScreen to determine the screen size!



  • @Christian-Ehrlicher
    How will knowing the screen size help this user if he want to know the central widget size?


  • Lifetime Qt Champion

    @JonB said in sometimes qt was ended forcefully, weird.:

    if he want to know the central widget size?

    my app is fullscreen always

    so...



  • @Christian-Ehrlicher
    So? :) The central widget is some widget in the full screen app, but there can be all sorts round the edges. So the widget itself won't be the full screen. I'm thinking like the central widget area in a QMainWindow. I guess you're not?


  • Lifetime Qt Champion

    @JonB said in sometimes qt was ended forcefully, weird.:

    I guess you're not?

    I don't see a need for all the calculation at all - using a proper layout will make all those stuff superfluous.



  • @Christian-Ehrlicher said in sometimes qt was ended forcefully, weird.:

    I don't see a need for all the calculation at all - using a proper layout will make all those stuff superfluous.

    I agree! But it wasn't the question :)



  • @all,
    thank you so much for your time. sorry i wasn't clear about my requirement. what i care is the centrawidget size when QMainWindow is in fullscreen.

    again, i am posting the code with the erros,

    MainWindow::MainWindow(QWidget *parent)
        : QMainWindow(parent)
        , ui(new Ui::MainWindow)
    {
        ui->setupUi(this);
        showFullScreen();
    
        ui->dockWidget->setWidget(ui->tableWidget);
    
        ui->line1->hide();
    }
    
    
    void MainWindow::resizeEvent(QResizeEvent *event)
    {
        QMainWindow::resizeEvent(event);
    
        // Your code here.
        static bool bOnlyOnce = true;
        if (bOnlyOnce && isVisible())
        {
            bOnlyOnce = false;
            
            memory allocation, file reading and data initialization
    
            QFileInfo fileInfo(QCoreApplication::applicationFilePath()); 
            if ()
                QMessageBox::information(this, "title", QString("%1").arg(fileInfo.size())); //***sometimes crash here ???
        
            get the size of centralwidget, prepare drawing environment
    
            ui->statusbar->addWidget(ui->lable1, 40); //***sometimes it crash here if i comment out QMessageBox line above*** ???
        }
    }
    

    now the QMessageBox line would crash sometimes(qtcreator says the process was ended forcefully), and if i comment out this line, the ui->statusbar line sometimes would report "Trying to construct an instance of an invalid type, type id: 1061" and the process was ended forcefully.
    i haven't found any rules about "sometimes", sometimes it will crash several times continuously when i press CTRL+R, but sometimes the program works fine.
    And it's working alwasys fine if i press F5 to debug the app.

    In addition, in the old backup version, it's working always fine. this version just has more functions after the crash point.


  • Lifetime Qt Champion

    As @J-Hilk already told you 4 days ago - don't create an eventloop in a resize event!



  • @Christian-Ehrlicher
    thanks, i am not 100% sure about that answer, but i will try another solution. now the question is, where do i get the size of centralwidget automatically when QMainWindows is in fullscreen (showFullScreen() in c'tor)? i tried showEvent and C'tor but failed to get the right size.



  • @circle
    I admit I have not tried it but I would have expected it in showEvent() as we discussed earlier. Could you show where you put what code for that?


  • Lifetime Qt Champion

    I still don't understand the need to know the size of a widget at all - your code you showed us doesn't do anything with it. Use layouts...



  • @JonB
    qDebug to print the line and ui->centralwidget->width()



  • @Christian-Ehrlicher
    well, i am supposed to do some complicated drawings as the background based on the size of centralwidget, there maybe several different solutions and i never use layouts before. those code was migrated from windows.



  • @circle said in sometimes qt was ended forcefully, weird.:

    qDebug to print the line and ui->centralwidget->width()

    Can you show just the few lines of code where you do this? What method it's in and where you do it.


  • Lifetime Qt Champion

    @circle said in sometimes qt was ended forcefully, weird.:

    i am supposed to do some complicated drawings as the background

    You can't draw on a widget outside the paintEvent()...



  • @JonB said in sometimes qt was ended forcefully, weird.:

    @circle said in sometimes qt was ended forcefully, weird.:

    qDebug to print the line and ui->centralwidget->width()

    Can you show just the few lines of code where you do this? What method it's in and where you do it.

    void MainWindow::showEvent(QShowEvent * event)
    {
        QMainWindow::showEvent(event);
        qDebug() << "w: " << ui->centralwidget->width() << "h: " << ui->centralwidget->height();
    }
    


  • @Christian-Ehrlicher said in sometimes qt was ended forcefully, weird.:

    @circle said in sometimes qt was ended forcefully, weird.:

    i am supposed to do some complicated drawings as the background

    You can't draw on a widget outside the paintEvent()...

    maybe paintEvent() is a better way, but i'm using eventFilter() instead (i am new to qt), more specifically, most of drawings are in the memory, such as qpixmap, qpainterpath, and they are working fine.


  • Lifetime Qt Champion

    But not in your example you show above - there you do a lot of stuff which doesn't belong to a resize (or paint)Event. I gave you a lot of examples on how to change it. Giving up.


Log in to reply