RE: GRUB 1.95 cannot read the ufs filesystem

[Hitoshi Ozeki]

Hello, all.

        Johan Rydberg wrote:
        > GRUB 1.95 cannot read the ufs filesystem.
        > This patch fixes this problem, and make it a bit better.
        Could you maybe point out the problem for us?  
        Personally, I think the use of pointers make the code more complex.

At first, When we use the 'strcmp' for the purpose of comparision C-strings,
It requires to terminate with the NIL sentry('\0').

----begin-------------------------------
grub_ufs_find_file (struct grub_ufs_data *data, const char *path)
{
  char fpath[grub_strlen (path)];  <-- not enough.
  char *name = fpath;
  char *next;
  unsigned int pos = 0;
  int dirino;
  
  grub_strncpy (fpath, path, grub_strlen (path));   <--without NIL.
----end---------------------------------

>From the manpage of 'strncpy(3)':
       The  strcpy()  function  copies the string pointed to by src
(including
       the terminating `\0' character) to the array pointed to by  dest.
The
       strings  may not overlap, and the destination string dest must be
large
       enough to receive the copy.

       The strncpy() function is similar, except that not more than n bytes
of
       src  are copied. Thus, if there is no null byte among the first n
bytes
       of src, the result will not be null-terminated.

At second, However we have the filepath in 'const char* path',
The original code copies the filepath. It wastes the processor time.

If you think the use of pointers makes complex,
I recommend to use array and index.

In addition, ufs filesystem has no label.

----begin-------------------------------
static struct grub_fs grub_ufs_fs =
  {
    .name = "ufs",
    .dir = grub_ufs_dir,
    .open = grub_ufs_open,
    .read = grub_ufs_read,
    .close = grub_ufs_close,
    .label = grub_ufs_label,
    .next = 0
  };
----end---------------------------------

The '.label' should set to 0.
On the original code, The 'label' function returns the invalid pointer,
so the 'ls -l' command gets wrong.

-- 
Regard,
 Hitoshi Ozeki