Project

General

Profile

Actions
Copy link

Bug #5712

closed

System Patches package: system_patches_edit.php - Patch contents is cleared when error happens

Added by Renato Botelho almost 9 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Renato Botelho
Category:
System Patches
Target version:
pfSense - 2.3
Start date:
12/29/2015
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
Affected Version:
2.3
Affected Plus Version:
Affected Architecture:

Description

Add a new patch, then fill 'patch contents' with a patch and 'URL/Commit ID' with invalid data then press 'Save'.

It'll report error and remove content from 'patch contents'

Actions
Copy link
 #1

Updated by Anonymous almost 9 years ago

  • Subject changed from System Patches: Patch contents is cleared when error happens to System Patches package: system_patches_edit.php - Patch contents is cleared when error happens
  • Status changed from New to Assigned
  • Assignee changed from Anonymous to Jim Pingle

It would appear that this is due to the patch being base64 encoded when saved to the config (no error) but being plain text when the page reloads after an error.

This is a simple way to fix it:

system_patches_edit.php line 184:

$patchtext = new Form_Textarea(
    'patch',
    'Patch Contents',
    $input_errors ? $pconfig['patch'] : base64_decode($pconfig['patch'])
);

Actions
Copy link
 #2

Updated by Jim Pingle almost 9 years ago

  • Status changed from Assigned to Feedback

Fix committed

Actions
Copy link
 #3

Updated by Anonymous almost 9 years ago

  • Assignee changed from Jim Pingle to Renato Botelho
Actions
Copy link
 #4

Updated by Anonymous almost 9 years ago

  • Status changed from Feedback to Resolved
Actions
Copy link
 #5

Updated by Chris Buechler over 8 years ago

  • Affected Version set to 2.3
Actions
Copy link

Also available in: Atom PDF