On Tue, Nov 05, 2013 at 08:39:08AM +0100, Sascha Hauer wrote: > On Mon, Nov 04, 2013 at 11:21:42PM +0100, Alexander Aring wrote: > > Hi Sascha, > > > > On Mon, Nov 04, 2013 at 03:04:26PM +0100, Sascha Hauer wrote: > > .... > > > > > > - entry->me.display = asprintf("%-20s %-20s %s", devname, hwdevname, > > > + entry->me.display = asprintf("%-20s %-20s %s", > > > + devname ? devname : "", > > > + hwdevname ? hwdevname : "", > > > blspec_entry_var_get(entry, "title")); > > > + > > > free(devname); > > > free(hwdevname); > > > > > > @@ -277,7 +281,7 @@ static int blspec_scan_cdev(struct blspec *blspec, struct cdev *cdev) > > > if (IS_ERR(rootpath)) > > > return PTR_ERR(rootpath); > > > > > > - ret = blspec_scan_directory(blspec, rootpath, cdev); > > > + ret = blspec_scan_directory(blspec, rootpath); > > > > > > return ret; > > unnecessary assign of ret here, but it's not important... > > Changed anyway. > oh neat, Thanks. _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox