compat/linprocfs

[Roman Divacky]

On Sat, Nov 24, 2007 at 05:44:26PM -0700, John Hein wrote:
> Roman Divacky wrote at 23:19 +0100 on Nov 24, 2007:
>  > in
>  > compat/linprocfs/linprocfs.c:linprocfs_doprocmaps() there's 
>  > 
>  >         unsigned int last_timestamp;
>  > 
>  > variable declaration, this variable is used only once in this function:
>  > 
>  >         if (last_timestamp + 1 != map->timestamp) {
>  > 
>  > ie. the map->timestamp is compared to some stack garbage + 1. this does not
>  > seem to be correct, the last_timestamp must be set somewhere..
>  > 
>  > comments?
> 
> It is set 6 lines earlier...
> 
> 		last_timestamp = map->timestamp;
> 		vm_map_unlock_read(map);
> 		error = uiomove(mebuffer, len, uio);
> 		vm_map_lock_read(map);
> 		if (error)
> 			break;
> 		if (last_timestamp + 1 != map->timestamp) {

sorry, mea culpa.. I have some local modifications :(

I apology for the noise