blob: 4c8f82aa3519fea3edd706042179618b37371f7a (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
|
From 0babf82f51561ac92a9b0dd56991d8de01930129 Mon Sep 17 00:00:00 2001
From: Vikram Bodireddy <vikram.bodireddy@intel.com>
Date: Sun, 5 Jul 2020 00:54:57 +0530
Subject: [PATCH] Fix for RedudancyPriority in item_updater
This fixes accessing RedudancyPriority property for the
activated image in item_updater. The downloaded image object
is not actually associated with RedudancyPriority before and
after activation. There exists no RedundancyPriority property
for downloaded image, accessing it causing a crash in
item_updater.
Tested: Tested for coredumps during Seamless firmware update.
Signed-off-by: Vikram Bodireddy <vikram.bodireddy@intel.com>
---
item_updater.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/item_updater.cpp b/item_updater.cpp
index af01e39..8d7bb82 100644
--- a/item_updater.cpp
+++ b/item_updater.cpp
@@ -655,8 +655,9 @@ void ItemUpdater::freeSpace(Activation& caller)
// Failed activations don't have priority, assign them a large value
// for sorting purposes.
auto priority = 999;
- if (iter.second.get()->activation() ==
- server::Activation::Activations::Active)
+ if ((iter.second.get()->activation() ==
+ server::Activation::Activations::Active)&&
+ iter.second->redundancyPriority.get())
{
priority = iter.second->redundancyPriority.get()->priority();
}
--
2.17.1
|
