Ticket #2988 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Suspected fread data loss bug in class-snoopy.php fix

Reported by: cschneid Assigned to: anonymous
Priority: normal Milestone:
Component: Administration Version: 2.0.4
Severity: normal Keywords:
Cc:

Description

I think the following should be done on line 1241 in wp-includes/class-snoopy.php:

 while (!feof($fp)) {
-  $file_content = fread($fp, filesize($file_name)); 
+  $file_content .= fread($fp, filesize($file_name)); 
 } 

i.e. concatenate the read data instead of just using the last part.

Change History

08/05/06 04:17:37 changed by ryan

  • version set to 2.0.4.
  • milestone set to 2.1.

Right you are. Luckily, traversing the loop more than once should be rare.

08/05/06 04:18:10 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

(In [4075]) Don't forget to concat. Props cschneid. fixes #2988

09/25/06 01:38:39 changed by ryan

(In [4222]) Don't forget to concat. Props cschneid. fixes #2988

09/25/06 04:08:36 changed by foolswisdom

  • milestone changed from 2.1 to 2.0.5.

11/30/06 19:41:51 changed by

  • milestone deleted.

Milestone 2.0.5 deleted