Bug 17311 - pem should be added as a searchable format to Photorec
Summary: pem should be added as a searchable format to Photorec
Status: RESOLVED WONTFIX
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 5
Hardware: All Linux
Priority: Normal enhancement
Target Milestone: ---
Assignee: Sander Lepik
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-12-08 00:38 CET by Kristoffer Grundström
Modified: 2015-12-12 19:24 CET (History)
1 user (show)

See Also:
Source RPM: photorec
CVE:
Status comment:


Attachments
My own pem-file (2.05 KB, text/x-csrc)
2015-12-08 00:40 CET, Kristoffer Grundström
Details
Updated filelist. (26.33 KB, text/x-csrc)
2015-12-08 00:40 CET, Kristoffer Grundström
Details
Criteria for a pem-file (7.45 KB, text/x-csrc)
2015-12-08 00:54 CET, Kristoffer Grundström
Details

Description Kristoffer Grundström 2015-12-08 00:38:11 CET
Description of problem: I noticed that the known file extension called .pem doesn't exist in Photorec so I took the opportunity to change file_list.c and then Makefile.am just like the cgsecurity wiki page tells you to if there's a file extension missing. I may have missed out to change some row in the two attached files, but here's my contribution anyway. 

Version-Release number of selected component (if applicable): 7.0

How reproducible: All the time.

Steps to Reproduce:
1. Open a Terminal/Console window.
2. Login as root.
3. Install testdisk and photorec by issuing urpmi testdisk photorec --auto
4. As root, type photorec and press Enter/Return or Execute button.
5. Choose to look in the entire hard drive and then change the files that you want to find and recover.
6. Go down to the letter p in the list of file types and you'll see that pem is in fact missing in the list.

Reproducible: 

Steps to Reproduce:
Kristoffer Grundström 2015-12-08 00:38:25 CET

CC: (none) => kristoffer.grundstrom1983

Comment 1 Kristoffer Grundström 2015-12-08 00:40:01 CET
Created attachment 7270 [details]
My own pem-file
Comment 2 Kristoffer Grundström 2015-12-08 00:40:46 CET
Created attachment 7271 [details]
Updated filelist.
Kristoffer Grundström 2015-12-08 00:41:08 CET

Source RPM: testdisk => photorec

Comment 3 Kristoffer Grundström 2015-12-08 00:54:08 CET
Created attachment 7272 [details]
Criteria for a pem-file
Comment 4 Kristoffer Grundström 2015-12-08 01:02:33 CET
Please check all the possible errors I may have missed to fix.

For the last attachment I used the file_png.c as a template so that's why it has some rows that's not supposed to be there.

I didn't know how to delete the unrelated rows so I left them to be changed by someone else.
Comment 5 Samuel Verschelde 2015-12-08 10:04:11 CET
Assigning to testdisk maintainer.

Assignee: bugsquad => mageia

Comment 6 Sander Lepik 2015-12-12 19:24:46 CET
I'd rather see you file a bug in upstream. They can include it and this way Mageia will get it as well. Adding such patches downstream doesn't seem practical.

Status: NEW => RESOLVED
Resolution: (none) => WONTFIX


Note You need to log in before you can comment on or make changes to this bug.