>> > > @@ -351,12 +351,17 @@ int cifs_get_inode_info_unix(struct inode **pinode,
>> > >                       cFYI(1, "CIFSCheckMFSymlink: %d", tmprc);
>> > >       }
>> > >
>> > > +     if (strstr(full_path, ":"))
>> > > +             fattr.cf_flags |= S_PRIVATE;
>> > > +
>> > >       if (*pinode == NULL) {
>> > >               /* get new inode */
>> > >               cifs_fill_uniqueid(sb, &fattr);
>> > >               *pinode = cifs_iget(sb, &fattr);
>> > >               if (!*pinode)
>> > >                       rc = -ENOMEM;
>> > > +             if (strstr(full_path, ":"))
>> > > +                     (*pinode)->i_flags |= S_PRIVATE;

^ On -ENOMEM you dereference a null pointer, don't you?  Same thing in
cifs_get_inode_info().


-- 
Peace and Blessings,
-Scott.
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to