diff options
| author | Stephen Warren <swarren@nvidia.com> | 2012-10-08 07:45:54 +0000 |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2012-10-17 07:59:09 -0700 |
| commit | 71bba424adcfa8c44100dee0fd139cc057eace65 (patch) | |
| tree | 1f0088c14448eec8312c32ef34a702023c6875b5 /disk/part.c | |
| parent | 8af4472a775e5719127cd109cd5e7f2bde470131 (diff) | |
| download | u-boot-71bba424adcfa8c44100dee0fd139cc057eace65.zip u-boot-71bba424adcfa8c44100dee0fd139cc057eace65.tar.gz u-boot-71bba424adcfa8c44100dee0fd139cc057eace65.tar.bz2 | |
disk: get_device_and_partition() return value fixes
When no valid partitions are found, guarantee that we return -1. This
most likely already happens, since the most recent get_partition_info()
will have returned an error. However, it's best to be explicit.
Remove an unnecessary assignment of ret=0 in the success case; this value
is over-written with the processed partition ID later.
Signed-off-by: Stephen Warren <swarren@nvidia.com>
Diffstat (limited to 'disk/part.c')
| -rw-r--r-- | disk/part.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/disk/part.c b/disk/part.c index 3022969..a0c77dd 100644 --- a/disk/part.c +++ b/disk/part.c @@ -624,9 +624,9 @@ int get_device_and_partition(const char *ifname, const char *dev_part_str, */ if (p == MAX_SEARCH_PARTITIONS + 1) *info = tmpinfo; - ret = 0; } else { printf("** No valid partitions found **\n"); + ret = -1; goto cleanup; } } |