|
 |
36520b |
From bc11451222cc77d8c1b4e752167adabd3c7f64c9 Mon Sep 17 00:00:00 2001
|
|
|
36520b |
From: Peter Jones <pjones@redhat.com>
|
|
|
36520b |
Date: Thu, 21 Jun 2018 13:33:26 -0400
|
|
|
36520b |
Subject: [PATCH 20/39] efiboot: Make the device node skipping code pass
|
|
|
36520b |
coverity.
|
|
|
36520b |
|
|
|
36520b |
Signed-off-by: Peter Jones <pjones@redhat.com>
|
|
|
36520b |
---
|
|
|
36520b |
src/linux.c | 31 ++++++++++++++++++-------------
|
|
|
36520b |
1 file changed, 18 insertions(+), 13 deletions(-)
|
|
|
36520b |
|
|
|
36520b |
diff --git a/src/linux.c b/src/linux.c
|
|
|
36520b |
index f919dee5b67..6d20c2dbe25 100644
|
|
|
36520b |
--- a/src/linux.c
|
|
|
36520b |
+++ b/src/linux.c
|
|
|
36520b |
@@ -308,7 +308,8 @@ struct device HIDDEN
|
|
|
36520b |
{
|
|
|
36520b |
struct device *dev;
|
|
|
36520b |
char *linkbuf = NULL, *tmpbuf = NULL;
|
|
|
36520b |
- unsigned int i, n = 0;
|
|
|
36520b |
+ int i = 0;
|
|
|
36520b |
+ unsigned int n = 0;
|
|
|
36520b |
int rc;
|
|
|
36520b |
|
|
|
36520b |
size_t nmemb = (sizeof(dev_probes)
|
|
|
36520b |
@@ -432,9 +433,11 @@ struct device HIDDEN
|
|
|
36520b |
int last_successful_probe = -1;
|
|
|
36520b |
|
|
|
36520b |
debug("searching for device nodes in %s", dev->link);
|
|
|
36520b |
- for (i = 0; dev_probes[i] && dev_probes[i]->parse; i++) {
|
|
|
36520b |
+ for (i = 0;
|
|
|
36520b |
+ dev_probes[i] && dev_probes[i]->parse && *current;
|
|
|
36520b |
+ i++) {
|
|
|
36520b |
struct dev_probe *probe = dev_probes[i];
|
|
|
36520b |
- ssize_t pos;
|
|
|
36520b |
+ int pos;
|
|
|
36520b |
|
|
|
36520b |
if (!needs_root &&
|
|
|
36520b |
(probe->flags & DEV_PROVIDES_ROOT)) {
|
|
|
36520b |
@@ -471,24 +474,26 @@ struct device HIDDEN
|
|
|
36520b |
debug("dev_probes[i+1]: %p dev->interface_type: %d\n",
|
|
|
36520b |
dev_probes[i+1], dev->interface_type);
|
|
|
36520b |
if (dev_probes[i+1] == NULL && dev->interface_type == unknown) {
|
|
|
36520b |
- int new_pos = 0;
|
|
|
36520b |
- rc = sscanf(current, "%*[^/]/%n", &new_pos);
|
|
|
36520b |
+ pos = 0;
|
|
|
36520b |
+ rc = sscanf(current, "%*[^/]/%n", &pos;;
|
|
|
36520b |
if (rc < 0) {
|
|
|
36520b |
- efi_error(
|
|
|
36520b |
- "Cannot parse device link segment \"%s\"",
|
|
|
36520b |
- current);
|
|
|
36520b |
+slash_err:
|
|
|
36520b |
+ efi_error("Cannot parse device link segment \"%s\"", current);
|
|
|
36520b |
goto err;
|
|
|
36520b |
}
|
|
|
36520b |
+
|
|
|
36520b |
+ while (current[pos] == '/')
|
|
|
36520b |
+ pos += 1;
|
|
|
36520b |
+
|
|
|
36520b |
+ if (!current[pos])
|
|
|
36520b |
+ goto slash_err;
|
|
|
36520b |
+
|
|
|
36520b |
debug("Cannot parse device link segment \"%s\"", current);
|
|
|
36520b |
debug("Skipping to \"%s\"", current + pos);
|
|
|
36520b |
debug("This means we can only create abbreviated paths");
|
|
|
36520b |
- if (rc < 0)
|
|
|
36520b |
- goto err;
|
|
|
36520b |
- if (new_pos == 0)
|
|
|
36520b |
- goto err;
|
|
|
36520b |
dev->flags |= DEV_ABBREV_ONLY;
|
|
|
36520b |
i = last_successful_probe;
|
|
|
36520b |
- current += new_pos;
|
|
|
36520b |
+ current += pos;
|
|
|
36520b |
}
|
|
|
36520b |
}
|
|
|
36520b |
|
|
|
36520b |
--
|
|
|
36520b |
2.17.1
|
|
|
36520b |
|