It looks completely harmless, but there's an inconsistency in
linux/proc/fs.
Do a: grep "inode->i_gid = p" *.c and you will see this field
set to
two different things by routines attempting the same sort of thing
(proc_Read_inode and fill_inode).
In one place it is set to p->egid and the other, p->gid. Surely one is
wrong.
Cheers
Chris