Message ID | 1320746280-31710-1-git-send-email-armbru@redhat.com |
---|---|
State | New |
Headers | show |
On 11/08/2011 03:58 AM, Markus Armbruster wrote: > g_strdup() can't fail, remove assertion. Assert its argument can't be > null, because that's not obvious (add_boot_device_path() ensures it). > > Signed-off-by: Markus Armbruster<armbru@redhat.com> Applied. Thanks. Regards, Anthony Liguori > --- > vl.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/vl.c b/vl.c > index 641629b..f169aac 100644 > --- a/vl.c > +++ b/vl.c > @@ -915,8 +915,8 @@ char *get_boot_devices_list(uint32_t *size) > } else if (devpath) { > bootpath = devpath; > } else { > + assert(i->suffix); > bootpath = g_strdup(i->suffix); > - assert(bootpath); > } > > if (total) {
diff --git a/vl.c b/vl.c index 641629b..f169aac 100644 --- a/vl.c +++ b/vl.c @@ -915,8 +915,8 @@ char *get_boot_devices_list(uint32_t *size) } else if (devpath) { bootpath = devpath; } else { + assert(i->suffix); bootpath = g_strdup(i->suffix); - assert(bootpath); } if (total) {
g_strdup() can't fail, remove assertion. Assert its argument can't be null, because that's not obvious (add_boot_device_path() ensures it). Signed-off-by: Markus Armbruster <armbru@redhat.com> --- vl.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)