diff --git a/checkin_notes b/checkin_notes
index 37351efb94..e1cee3133f 100644
--- a/checkin_notes
+++ b/checkin_notes
@@ -3951,3 +3951,12 @@ David 15 Apr 2009
translation.inc
samples/example_app/
uc2.cpp
+
+Davaid 16 Apr 2009
+ - client: another try at fixing the above crash
+ - web: don't show plan class in separate column in apps list; fixes #874
+
+ client/
+ app_control.cpp
+ html/user/
+ apps.php
diff --git a/client/app_control.cpp b/client/app_control.cpp
index 4750f46757..8b532e1cb5 100644
--- a/client/app_control.cpp
+++ b/client/app_control.cpp
@@ -413,7 +413,7 @@ void ACTIVE_TASK_SET::send_heartbeats() {
atp = active_tasks[i];
if (!atp->process_exists()) continue;
if (!atp->app_client_shm.shm) continue;
- sprintf(buf, ""
+ snprintf(buf, sizeof(buf), ""
"%e"
"%e",
atp->procinfo.working_set_size, ar
diff --git a/html/user/apps.php b/html/user/apps.php
index 7bec4102a9..e77da6a974 100644
--- a/html/user/apps.php
+++ b/html/user/apps.php
@@ -46,7 +46,6 @@ foreach ($apps as $app) {
".tra("Platform")." |
".tra("Version")." |
- ".tra("Plan class")." |
".tra("Installation time")." |
";
@@ -65,23 +64,25 @@ foreach ($apps as $app) {
}
foreach($avs as $av) {
if ($av->deprecated) continue;
- $y = pretty_time_str($av->create_time);
+ $create_time_f = pretty_time_str($av->create_time);
if ($xml) {
echo " \n";
echo " $platform->name\n";
echo " $platform->user_friendly_name\n";
echo " $av->version_num\n";
echo " $av->plan_class\n";
- echo " $y\n";
+ echo " $create_time_f\n";
echo " $av->create_time\n";
echo " \n";
} else {
- $x = sprintf("%0.2f", $av->version_num/100);
+ $version_num_f = sprintf("%0.2f", $av->version_num/100);
+ if ($av->plan_class) {
+ $version_num_f .= " ($av->plan_class)";
+ }
echo "
$platform->user_friendly_name |
- $x |
- $av->plan_class |
- $y |
+ $version_num_f |
+ $create_time_f |
";
}