Sign me up Login

Details about package "kpdftool"

Name: kpdftool
Uploader: Simone Rossetto <simros85@gmail.com> (Debian QA page)
Description: kpdftool - GUI to manipulate PDF and PS files using GhostScript and ImageMag

Package versions

Version 0.23.1-1

Information

Version: 0.23.1-1
Uploaded: 2012-03-03 09:14
Source package: http://mentors.debian.net/debian/pool/main/k/kpdftool/kpdftool_0.23.1-1.dsc
Section: graphics
Priority: optional
Closes bugs: 644726

QA information

Comments

  1. The packaging itself seems to be very good and complete. The upstream changelog can be found in the README.
    
    However, there is a serious bug in kpdftool:
     
     - It uses hard-coded temporary files in /tmp which makes it unusable on multi-user systems and includes security and privacy issues. Using e.g. QTemporaryFile or creating a temporary subdirectory (like mkdtemp) could be the solution.
     The application should just delete its own temporary files. system("rm -rf /tmp/kpdftool*.png") is definitely the wrong way => QFile::remove
    
    IMHO, the real trivial "recover contents" feature is not sufficient that the package depends on kword. I would use "recommend" or just "suggest". inkscape can open pdf files, too.
    
    On my wishlist would be the possibility to configure where output files are written to for all operations, a check if the output file is identical to the input file, and some documentations what each operation does exactly. For example, "protect pdf" does not protect the file by setting some passwords/permissions (like it can be done by pdfchain) as one could expect, but rasterize everything.
    
    Needs work Johann Felix Soden at 2012-03-29 23:09:42.561777
  2. Hi Johann, thanks for your review. I'll send the whole comment to the upstream author and we'll work to fix those problem.
    Needs work Simone Rossetto at 2012-04-05 19:51:59.978569