summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | releasing package apt version 1.1~exp4Michael Vogt2014-10-081-1/+1
| |
* | prepare 1.1~exp4Michael Vogt2014-10-081-0/+80
| |
* | Fix ServerMethod::FindMaximumObjectSizeInQueue()Michael Vogt2014-10-081-1/+1
| | | | | | | | Git-Dch: ignore
* | pkgAcqArchive::QueueNext(): change owner/permission of DestFileMichael Vogt2014-10-081-1/+1
| | | | | | | | | | The code was using FinalFile before but we only test the existance of DestFile so we use that instead.
* | Merge branch 'debian/sid' into debian/experimentalMichael Vogt2014-10-081-0/+28
|\| | | | | | | | | Conflicts: debian/changelog
| * releasing package apt version 1.0.9.2Michael Vogt2014-10-021-0/+28
| |
| * apt-get: Create the temporary downloaded changelog inside tmpdirGuillem Jover2014-10-021-1/+1
| | | | | | | | | | | | | | | | The code is creating a secure temporary directory, but then creates the changelog alongside the tmpdir in the same base directory. This defeats the secure tmpdir creation, making the filename predictable. Inject a '/' between the tmpdir and the changelog filename.
| * debian/rules: add hardening=+allMichael Vogt2014-10-011-3/+6
| | | | | | | | | | | | | | Because of dpkg-buildflags we already get most of the hardening features, +all adds -fPIE and ld -z now Thanks: Simon Ruderich, Markus Waldeck
* | Merge remote-tracking branch 'mvo/feature/expected-size' into ↵Michael Vogt2014-10-0815-39/+155
|\ \ | | | | | | | | | debian/experimental
| * | Fix http pipeline messup detectionMichael Vogt2014-10-082-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | The Maximum-Size protection breaks the http pipeline reorder code because it relies on that the object got fetched entirely so that it can compare the hash of the downloaded data. So instead of stopping when the Maximum-Size of the expected item is reached we only stop when the maximum size of the biggest item in the queue is reached. This way the pipeline reoder code keeps working.
| * | Merge remote-tracking branch 'donkult/feature/acq-trans' into ↵Michael Vogt2014-10-084-11/+26
| |\ \ | | | | | | | | | | | | feature/expected-size
| * | | Send "Fail-Reason: MaximumSizeExceeded" from the methodMichael Vogt2014-10-076-6/+21
| | | | | | | | | | | | | | | | | | | | Communicate the fail reason from the methods to the parent and Rename() failed files.
| * | | Add new Acquire::MaxReleaseFileSize=10*1000*1000 optionMichael Vogt2014-10-073-31/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This option controls the maximum size of Release/Release.gpg/InRelease files. The rational is that we do not know the size of these files in advance and we want to protect against a denial of service attack where someone sends us endless amounts of data until the disk is full (we do know the size all other files (Packages/Sources/debs)).
| * | | Merge branch 'feature/acq-trans' into feature/expected-sizeMichael Vogt2014-10-0743-1146/+2043
| |\ \ \
| * | | | make expected-size a maximum-size check as this is what we want at this pointMichael Vogt2014-10-0710-19/+31
| | | | |
| * | | | fix test-cve-2013-1051-InRelease-parsing (fails now in the method)Michael Vogt2014-10-071-1/+1
| | | | |
| * | | | add ftp expected size checkMichael Vogt2014-10-073-5/+8
| | | | |
| * | | | methods/https.cc: use File->Tell() here tooMichael Vogt2014-10-071-2/+1
| | | | |
| * | | | make http size check workMichael Vogt2014-10-06308-81287/+92461
| |\ \ \ \
| * | | | | make https honor ExpectedSize as wellMichael Vogt2014-08-262-1/+8
| | | | | |
| * | | | | Pass ExpectedSize to tthe backend methodMichael Vogt2014-08-267-3/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This ensures that we can stop downloading if the server send too much data by accident (or by a malicious attempt)
* | | | | | Merge remote-tracking branch 'mvo/feature/acq-trans' into debian/experimentalMichael Vogt2014-10-0845-1156/+2068
|\ \ \ \ \ \ | | |_|_|/ / | |/| | | |
| * | | | | fix http-pipeline-messup testcaseDavid Kalnischkies2014-10-081-8/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test generates failures if the created deb files have the same size, so we try a little harder to avoid having the same size for them. Git-Dch: Ignore
| * | | | | do not show IP in output of testcasesDavid Kalnischkies2014-10-072-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On travis-ci connect.cc detects a rotation, triggering it store the IP which is later appended to the error message, which is all nice and great if we deal with a real server, but in the testcases it just triggers failures as strings do not match. Git-Dch: Ignore
| * | | | | don't show ErrorText for Ign by defaultDavid Kalnischkies2014-10-072-2/+7
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some distributions (or repositories) do not have as much "Ign-discipline" as I would like to, so that could be pretty distracting for our users if enabled by default. It is handy for testcases though. Git-Dch: Ignore
| * | | | Merge remote-tracking branch 'upstream/debian/experimental' into ↵Michael Vogt2014-10-0739-536/+901
| |\ \ \ \ | | | |/ / | | |/| | | | | | | | | | | | | | | | | | | | | | feature/acq-trans Conflicts: apt-pkg/acquire-item.cc
| * | | | UpperCase some functions for consistencyMichael Vogt2014-10-071-25/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | changeOwnerAndPermissionOfFile->ChangeOwnerAndPermissionOfFile preparePartialFile->GetPartialFileName preparePartialFileFromURI->GetPartialFileNameFromURI Git-Dch: ignore
| * | | | display errortext for all Err as well as Ign logsDavid Kalnischkies2014-10-078-101/+108
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | consistently using Item::Failed in all specializec classes helps setting up some information bits otherwise unset, so some errors had an empty reason as an error. Ign is upgraded to display the error message we ignored to further help in understanding what happens.
| * | | | use _apt:root only for partial directoriesDavid Kalnischkies2014-10-077-79/+137
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using a different user for calling methods is intended to protect us from methods running amok (via remotely exploited bugs) by limiting what can be done by them. By using root:root for the final directories and just have the files in partial writeable by the methods we enhance this in sofar as a method can't modify already verified data in its parent directory anymore. As a side effect, this also clears most of the problems you could have if the final directories are shared without user-sharing or if these directories disappear as they are now again root owned and only the partial directories contain _apt owned files (usually none if apt isn't running) and the directory itself is autocreated with the right permissions.
| * | | | ensure partial dirs are 0700 and owned by _apt:rootDavid Kalnischkies2014-10-078-55/+81
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reworks the API involved in creating and setting up the fetcher to be a bit more pleasent to look at and work with as e.g. an empty string for no lock isn't very nice. With the lock we can also stop creating all our partial directories "just in case". This way we can also be a bit more aggressive with the partial directory itself as with a lock, we know we will gone need it.
| * | | | fixup foldmarkers in acquire-item.ccDavid Kalnischkies2014-10-071-91/+51
| | | | | | | | | | | | | | | | | | | | Git-Dch: Ignore
| * | | | fix warningsMichael Vogt2014-10-061-3/+3
| | | | |
| * | | | rename StopAuthentication -> CheckStopAuthentication and make it protectedMichael Vogt2014-10-062-9/+11
| | | | |
| * | | | fix testMichael Vogt2014-10-061-2/+1
| | | | |
| * | | | rename AuthDone() -> CheckAuthDone()Michael Vogt2014-10-062-6/+4
| | | | |
| * | | | cleanup pkgAcq*::Failed()Michael Vogt2014-10-063-73/+55
| | | | |
| * | | | add missing TransactionStageCopy() in pkgAcqDiffIndex::Done()Michael Vogt2014-10-061-12/+5
| | | | |
| * | | | Rework pkgAcqMeta{Index,Sig,ClearSig}::Done() for readabilityMichael Vogt2014-10-063-213/+149
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move common code out but do not use subclassing for ::Done to make it easier to understand what each class is doing when its done
| * | | | fix incorrect docstrings for ↵Michael Vogt2014-10-061-2/+2
| | | | | | | | | | | | | | | | | | | | AcqMetaBase::TransactionStageRemoval/AcqMetaBase::TransactionStageCopy
| * | | | add new "SetActiveSubprocess()Michael Vogt2014-10-062-46/+25
| | | | |
| * | | | update testMichael Vogt2014-10-061-3/+7
| | | | |
| * | | | really do not download Release if InRelease does not verifyMichael Vogt2014-10-031-20/+24
| | | | |
| * | | | cleanup around pkgAcqMetaSig and improved testsMichael Vogt2014-10-023-52/+76
| | | | |
| * | | | add a bunch of docstrings etcMichael Vogt2014-10-022-14/+40
| | | | |
| * | | | fix crashMichael Vogt2014-10-022-11/+14
| | | | |
| * | | | donkults fixesMichael Vogt2014-10-023-41/+38
| | | | |
| * | | | Cleanup pkgAcqIndexMichael Vogt2014-10-022-135/+171
| | | | |
| * | | | refactor and add pkgAcqIndex::ValidateFile()Michael Vogt2014-10-012-25/+39
| | | | |
| * | | | fix leftover files from Acquire::GzipIndexMichael Vogt2014-10-012-8/+23
| | | | |
| * | | | hack around test-apt-update-unauth failureMichael Vogt2014-10-012-6/+24
| | | | |