[Libguestfs] [PATCH 2/3] inspection: Add support for CoreOS

Nikos Skalkotos skalkoto at grnet.gr
Tue Jun 2 15:43:46 UTC 2015


In inspect-fs-unix.c:1197 in add_fstab_entry() then too.


On 02/06/15 18:39, Richard W.M. Jones wrote:
> On Tue, Jun 02, 2015 at 06:37:06PM +0300, Nikos Skalkotos wrote:
>> Sorry, I don't get it. In inspect.c line 67 you have this:
>>
>>   for (fs = fses; *fs; fs += 2) {
>>     if (guestfs_int_check_for_filesystem_on (g, *fs)) {
>>       guestfs_int_free_inspect_info (g);
>>       return NULL;
>>     }
>>   }
> Oh I see, yes that's a bug too.
>
>> I don't see the bug.
> I was looking at this other bug in src/inspect-fs.c:
>
>   static int
>   extend_fses (guestfs_h *g)
>   {
>     size_t n = g->nr_fses + 1;
>     struct inspect_fs *p;
>
>     p = realloc (g->fses, n * sizeof (struct inspect_fs));
>     if (p == NULL) {
>       perrorf (g, "realloc");
>       return -1;
>     }
>
> Should probably just call safe_realloc ..
>
> Rich.
>




More information about the Libguestfs mailing list