polipo index does not show all cache entries

Bug #988780 reported by Chris Bainbridge
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
polipo (Ubuntu)
Fix Released
Low
Rolf Leggewie

Bug Description

In indexDiskObjects() readdir returns many entries, then the entry "..", which causes dobjects to be set to 0x0 thus removing every entry that has already been enumerated and stored in dobjects.

Patch attached.

ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: polipo 1.0.4.1-1.1
ProcVersionSignature: Ubuntu 3.2.0-23.36-generic 3.2.14
Uname: Linux 3.2.0-23-generic x86_64
ApportVersion: 2.0.1-0ubuntu5
Architecture: amd64
Date: Thu Apr 26 10:51:51 2012
ProcEnviron:
 LANGUAGE=en_GB:en
 TERM=xterm
 PATH=(custom, user)
 LANG=en_GB.UTF-8
 SHELL=/bin/bash
SourcePackage: polipo
UpgradeStatus: No upgrade log present (probably fresh install)
mtime.conffile..etc.polipo.config: 2012-04-26T09:07:12.268208

Related branches

Revision history for this message
Chris Bainbridge (chris-bainbridge) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "polipo-index-fix.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Rolf Leggewie (r0lf) wrote :

Thank you for the report and patch. As far as I can see this has already been committed upstream in https://github.com/jech/polipo/commit/8e23f1493437b5470ce560e9ab4ad41c4a915835

Changed in polipo (Ubuntu):
assignee: nobody → Rolf Leggewie (r0lf)
status: New → Fix Committed
importance: Undecided → Low
Revision history for this message
Rolf Leggewie (r0lf) wrote :

For those of you who would like to test my wip packages before I upload them to Debian, you can get them for precise (and later) from my stable PPA: https://launchpad.net/~r0lf/+archive/stable

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package polipo - 1.1.1-1

---------------
polipo (1.1.1-1) unstable; urgency=medium

  * release newest upstream version 1.1.1. Closes: #746750
    - use regerror to improve error reporting. Closes: #585766
    - correct return value from processObject in error case. LP: #988780
    - resume interrupted file download properly. Closes: #517479
    - vastly improve stability. LP: #1055418, LP: #988780, LP: #1168593
      Closes: #684742, Closes: #307801, Closes: #432561, Closes: #646709
    - handle non-existent host properly. Closes: #655851
    - Don't revalidate objects >= CHUNK_SIZE that aren't in the disk cache.
      Closes: #610199
  * drop patches included upstream and rebase the rest
  * update watch file

 -- Rolf Leggewie <email address hidden> Fri, 06 Jun 2014 15:29:19 +0200

Changed in polipo (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.