[SoftwareUpdater] crash

[diver]

hrev51114.

SoftwareUpdater crashed when started with no network connection.

[perelandra]

Thanks for the report. This will be fixed in version 1.0.1. Strangely this only happened on x64.

diff --git a/src/apps/softwareupdater/UpdateManager.cpp b/src/apps/softwareupdater/UpdateManager.cpp
index 12fa3b0..5041380 100644
--- a/src/apps/softwareupdater/UpdateManager.cpp
+++ b/src/apps/softwareupdater/UpdateManager.cpp
@@ -243,8 +243,8 @@ UpdateManager::Warn(status_t error, const char* format, ...)
        char buffer[256];
        va_list args;
        va_start(args, format);
-       vfprintf(stderr, format, args);
        vsnprintf(buffer, sizeof(buffer), format, args);
+       fprintf(stderr, buffer);
        va_end(args);
 
        if (error == B_OK)

[korli]

@perelandra we usually leave the bug report open until the fix actually lands in git master with a hrev to be tested by the reporter.

[perelandra]

Replying to korli:

@perelandra we usually leave the bug report open until the fix actually lands in git master with a hrev to be tested by the reporter.

OK

[axeld]

Also, it's not a good idea to use fprintf() when you don't actually have any formatting to do. Use fputs() instead.

[perelandra]

In all the code I've browsed through I don't recall seeing fputs or puts used so that doesn't seem to be strictly enforced. I had to look those up to know what they do. How about using: fprintf(stderr, "%s", string) ?

[perelandra]

Fixed code in hrev51148, also removed the ability to "Continue anyway" when no network is detected since there relly is no point in doing so.