Bug 6150 - SVN fetching trusts certificates by default
Summary: SVN fetching trusts certificates by default
Status: NEW
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: configuration (show other bugs)
Version: unspecified
Hardware: x86 Multiple
: Low normal
Target Milestone: 6.2
Assignee: Richard Purdie
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2014-04-10 16:38 UTC by Zoltan Kuscsik
Modified: 2026-10-01 15:48 UTC (History)
5 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: Don't know


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Zoltan Kuscsik 2014-04-10 16:38:46 UTC
Default configuration for SVN fetching always trusts the server certificate:

meta/conf/bitbake.conf


This introduces some risks when using bitbake in a secure environment.
Comment 1 Richard Purdie 2014-04-13 20:33:20 UTC
This has been the case for a long time, users tend to find it a pain to need to verify certificates for all svn urls. The user or distro can override the options from bitbake.conf for "secure" setups should that be necessary. I can't see a pressing need to change this.
Comment 2 Zoltan Kuscsik 2014-04-13 21:25:35 UTC
My view is that the current configuration should not be used in any build configuration where security is a concern. There are many situations where user don't need or don't care about downloading the source code securely, however, having this as a default configuration creates a false sense of security.

Without certificate verification an attacker could replace code sequences that will be built in to the target system.
Comment 3 Richard Purdie 2014-04-22 15:05:06 UTC
I still don't see this to be a high priority issue however if you want to take it on and make it happen I have no objection, up to you really.
Comment 4 Zoltan Kuscsik 2015-11-20 05:39:44 UTC
I suggest to make it a warning. Sent a patch to the mailing list.

//Zoli
Comment 5 Richard Purdie 2020-01-23 16:18:05 UTC
We still do:
         ud.basecmd = d.getVar("FETCHCMD_svn") or "/usr/bin/env svn --non-interactive --trust-server-cert"
in lib/bb/fetch2/svn.py
Comment 6 Randy MacLeod 2020-01-23 16:40:35 UTC
FYI, the patch is here:
  http://lists.openembedded.org/pipermail/bitbake-devel/2015-November/016452.html

Zoltan, can you resend? If not, I'll get someone to test and submit the patch.
Comment 7 Richard Purdie 2022-01-27 16:09:31 UTC
Can we just drop the option from the commandline?