Menu ▾ ▴

#1632 MinGW-get, Crashes, {--verbose, -v}; 0.5-beta-20120416-1

IINR_-_Include_In_Next_Release
closed-fixed
Keith Marshall
mingw-get (5)
2012-04-23
2012-04-23
Melchior
No

When I attempted to use the either of the commands,
--verbose, -v
--verbose=N

mingw-get.exe crashes

I am using version,
mingw-get version 0.5-beta-20120416-1

PS:
I thought I'd take a moment to report it encase it was unknown to help out a bit ^_^

Discussion

  • Melchior

    Melchior - 2012-04-23

    Dr.Watson Mini Crash Dump 2nd

     
    drwtsn32_MinGW-Get_2_user.dmp

  • Melchior

    Melchior - 2012-04-23

    Dr.Watson Crash Log file

     
    drwtsn32_MinGW-Get.log

  • Keith Marshall

    Keith Marshall - 2012-04-23

    Thanks for the report.

    Sadly, the Dr. Watson dumps aren't much help, but I can reproduce a crash under Wine, only when I issue a malformed mingw-get command:

    ./mingw-get -v[v]

    *without* specifying the *required* action arguments. When the proper arguments are specified appropriately, as in:

    ./mingw-get -vv install mingw-get

    for example, -v, -vv, --verbose=N, etc. all WJFFM. However, mingw-get shouldn't segfault, (which is what is happening), when the proper arguments are omitted -- either, the GUI stub should be invoked, to advise how to get help on correct CLI usage, or a usage error should be diagnosed directly, by the CLI argument interpreter.

    Thanks again. This hadn't previously come to my attention. I will investigate further.

     

  • Melchior

    Melchior - 2012-04-23

    Your Most Welcome ^_^
    All I did was execute the
    --verbose, -v
    --verbose=N
    trying to up the Verbosity....

    PS: you mentioned Wine so I take it your run Linux o-O ^_^
    if you don't mind me asking what Distro are you running?

     

  • Melchior

    Melchior - 2012-04-23

    Ahh I see when I Specify an Action Command with the update action like so:
    .\MinGW-Get.exe --verbose=3 update

    it doesn't crash...
    Normally it would only display the:
    Update catalogue: Catalog Name
    line....

    it also displayed a
    Load catalogue:
    when I upped to verbosity with an update action command specified... lol

    I see what you mean... it shouldn't be crashing it should show a
    you didn't do that correctly output lol

    that should be a small enough fix... perhaps

    I'm happy I could help in any way ^_^

     

  • Melchior

    Melchior - 2012-04-23

    and when I use it with the upgrade, action

    It outputs

    loading each xml catalog file in addition to the
    upgrading/already up to date lines....

    loi :)
    nice

    I guess I was expecting it to remember my verbosity section for all the time I use the
    MinGW-Get.exe

     

  • Keith Marshall

    Keith Marshall - 2012-04-23
    • milestone: --> IINR_-_Include_In_Next_Release
    • status: open --> closed-fixed
     

  • Keith Marshall

    Keith Marshall - 2012-04-23

    Easy enough to fix: need to check for a NULL pointer when identifying the action to perform, decline to pass it to strlen()/strncmp(), and adjust the diagnostics to accommodate the additional abnormal case.

    I've also taken the opportunity to address a further abnormality in the same section of the code. Use of strncmp() supports the use of abbreviations for the action keyword, but ambiguous abbreviations were resolved on the basis of first checked succeeds. They should have been rejected, and will be from next release onward.

    Issue fixed, as noted, and committed to CVS; will incorporate in next release.

    FWIW, unrelated to this bug but to answer your query: yes, I do run Linux; currently LinuxMint-10, but I've previously used RedHat-6, Mandrake-7.2 and 8.1, SUSE-10, Ubuntu-6.06 and 8.04. All releases of mingw-get, to date, have been built on one or other of these distros, using a self-built GCC-3.4.5 cross-compiler, built from MinGW.org sources.

     

  • Melchior

    Melchior - 2012-04-24

    Eyy ^_^ WoW I very happy I could help out ^_^
    I was born '82=Year of the Dog so I tend to be kinda a fault finder of sorts ^_^,
    I guess I would make a good Beta tester maybe once in a while lol.

    I was wonder I don't mean to pester... any idea when the next release might be out?
    loi

    Your using String Compare? to.... compare it to what commands/actions are available...?
    just wondering.... I have an old Temperature Converter Program from my C Class days back 2001-2002...(college)
    I expanded it beyond what the class had done with it using Switches to select between the different conversion Paths.....

    PS:
    Linux Mint 10,... Cool I went and took a look at there site Cool looking.. they are up to version 12 now.... o-O :)
    they have a Gnome and a KDE version.....
    when I was checking out a distro called Ark Linux years (4-7yrs) i kept an eye on kde... :)

     

  • Keith Marshall

    Keith Marshall - 2012-04-26

    > any idea when the next release might be out?

    Already available, from earlier today.

    > Your using String Compare? to....

    http://mingw.cvs.sourceforge.net/viewvc/mingw/mingw-get/

     

  • Melchior

    Melchior - 2012-04-27

    Thx for the update ^_^
    I tried the update and Upgrade feature of current version today to update....
    There were no updates for MinGW-Get from this way...
    I guessing that wasn't or is updated separately...

    I'll get it manually