summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--usr.sbin/pkg_add/OpenBSD/Add.pm262
1 files changed, 154 insertions, 108 deletions
diff --git a/usr.sbin/pkg_add/OpenBSD/Add.pm b/usr.sbin/pkg_add/OpenBSD/Add.pm
index af07a008a66..7429cf6d0bf 100644
--- a/usr.sbin/pkg_add/OpenBSD/Add.pm
+++ b/usr.sbin/pkg_add/OpenBSD/Add.pm
@@ -1,5 +1,5 @@
# ex:ts=8 sw=4:
-# $OpenBSD: Add.pm,v 1.46 2005/12/05 13:39:28 espie Exp $
+# $OpenBSD: Add.pm,v 1.47 2006/02/09 09:42:10 espie Exp $
#
# Copyright (c) 2003-2004 Marc Espie <espie@openbsd.org>
#
@@ -63,75 +63,9 @@ sub validate_plist($$)
my $problems = 0;
my $pkgname = $plist->pkgname();
my $totsize = 0;
-
- my $extra = $plist->{extrainfo};
- if ($state->{cdrom_only} && ((!defined $extra) || $extra->{cdrom} ne 'yes')) {
- Warn "Package $pkgname is not for cdrom.\n";
- $problems++;
- }
- if ($state->{ftp_only} && ((!defined $extra) || $extra->{ftp} ne 'yes')) {
- Warn "Package $pkgname is not for ftp.\n";
- $problems++;
- }
-
- # check for collisions with existing stuff
my $colliding = [];
- for my $item (@{$plist->{items}}) {
- next unless $item->IsFile();
- my $fname = $destdir.$item->fullname();
- if (OpenBSD::Vstat::vexists($fname)) {
- push(@$colliding, $item);
- $problems++;
- next;
- }
- $totsize += $item->{size} if defined $item->{size};
- my $s = OpenBSD::Vstat::add($fname, $item->{size}, \$pkgname);
- next unless defined $s;
- if ($s->{ro}) {
- if ($state->{very_verbose} or ++($s->{problems}) < 4) {
- Warn "Error: ", $s->{dev},
- " is read-only ($fname)\n";
- } elsif ($s->{problems} == 4) {
- Warn "Error: ... more files can't be written to ",
- $s->{dev}, "\n";
- }
- $problems++;
- }
- if ($state->{forced}->{kitchensink} && $state->{not}) {
- next;
- }
- if ($s->avail() < 0) {
- if ($state->{very_verbose} or ++($s->{problems}) < 4) {
- Warn "Error: ", $s->{dev},
- " is not large enough ($fname)\n";
- } elsif ($s->{problems} == 4) {
- Warn "Error: ... more files do not fit on ",
- $s->{dev}, "\n";
- }
- $problems++;
- }
- }
- my $dest = installed_info($pkgname);
- my $dir = $plist->{dir};
- for my $i (info_names()) {
- if (-e "$dir/$i") {
- my $size = (stat _)[7];
- my $fname = "$dest/$i";
- my $s = OpenBSD::Vstat::add($fname, $size, \$pkgname);
- next unless defined $s;
- if ($s->{ro}) {
- Warn "Error: ", $s->{dev}, " is read-only ($fname)\n";
- $problems++;
- }
- if ($state->{forced}->{kitchensink} && $state->{not}) {
- next;
- }
- if ($s->avail() < 0) {
- Warn "Error: ", $s->{dev}, " is not large enough ($fname)\n";
- $problems++;
- }
- }
- }
+
+ $plist->visit('validate', $state, \$problems, $colliding, \$totsize, $plist->pkgname());
if (@$colliding > 0) {
require OpenBSD::CollisionReport;
@@ -187,6 +121,10 @@ use OpenBSD::Error;
my ($uidcache, $gidcache);
+sub validate
+{
+}
+
sub install
{
}
@@ -221,6 +159,26 @@ sub set_modes
}
}
+package OpenBSD::PackingElement::ExtraInfo;
+use OpenBSD::Error;
+
+sub validate
+{
+ my ($self, $state, $problems, $colliding, $totsize, $pkgname) = @_;
+
+ if ($state->{cdrom_only} && $self->{cdrom} ne 'yes') {
+ Warn "Package $pkgname is not for cdrom.\n";
+ $$problems++;
+ }
+ if ($state->{ftp_only} && $self->{ftp} ne 'yes') {
+ Warn "Package $pkgname is not for ftp.\n";
+ $$problems++;
+ }
+}
+
+package OpenBSD::PackingElement::NewAuth;
+use OpenBSD::Error;
+
sub add_entry
{
shift; # get rid of self
@@ -237,54 +195,59 @@ sub add_entry
}
}
-package OpenBSD::PackingElement::NewUser;
-use OpenBSD::Error;
-
-sub install
+sub validate
{
- my ($self, $state) = @_;
- my $user = $self->{name};
- print "adding user $user\n" if $state->{verbose};
- return if $state->{not};
+ my ($self, $state, $problems, $colliding, $totsize, $pkgname) = @_;
my $ok = $self->check();
if (defined $ok) {
if ($ok == 0) {
- Fatal "user $user does not match\n";
+ Warn $self->type(), " ", $self->{name},
+ " does not match\n";
+ $$problems++;
}
- } else {
- my $l=[];
- push(@$l, "-v") if $state->{very_verbose};
- $self->add_entry($l,
- '-u', $self->{uid},
- '-g', $self->{group},
- '-L', $self->{class},
- '-c', $self->{comment},
- '-d', $self->{home},
- '-s', $self->{shell});
- VSystem($state->{very_verbose}, '/usr/sbin/useradd', @$l, $user);
}
+ $self->{okay} = $ok;
}
-package OpenBSD::PackingElement::NewGroup;
-use OpenBSD::Error;
-
sub install
{
my ($self, $state) = @_;
- my $group = $self->{name};
- print "adding group $group\n" if $state->{verbose};
+ my $auth = $self->{name};
+ print "adding ", $self->type(), " $auth\n" if $state->{verbose};
return if $state->{not};
- my $ok = $self->check();
- if (defined $ok) {
- if ($ok == 0) {
- Fatal "group $group does not match\n";
- }
- } else {
- my $l=[];
- push(@$l, "-v") if $state->{very_verbose};
- $self->add_entry($l, '-g', $self->{gid});
- VSystem($state->{very_verbose}, '/usr/sbin/groupadd', @$l, $group);
- }
+ return if defined $self->{okay};
+ my $l=[];
+ push(@$l, "-v") if $state->{very_verbose};
+ $self->build_args($l);
+ VSystem($state->{very_verbose}, $self->command(),, @$l, $auth);
+}
+
+package OpenBSD::PackingElement::NewUser;
+
+sub command { '/usr/sbin/useradd' }
+
+sub build_args
+{
+ my ($self, $l) = @_;
+
+ $self->add_entry($l,
+ '-u', $self->{uid},
+ '-g', $self->{group},
+ '-L', $self->{class},
+ '-c', $self->{comment},
+ '-d', $self->{home},
+ '-s', $self->{shell});
+}
+
+package OpenBSD::PackingElement::NewGroup;
+
+sub command { '/usr/sbin/groupadd' }
+
+sub build_args
+{
+ my ($self, $l) = @_;
+
+ $self->add_entry($l, '-g', $self->{gid});
}
package OpenBSD::PackingElement::Sysctl;
@@ -317,6 +280,44 @@ use OpenBSD::Error;
use File::Basename;
use File::Path;
+sub validate
+{
+ my ($self, $state, $problems, $colliding, $totsize, $pkgname) = @_;
+ my $fname = $state->{destdir}.$self->fullname();
+ # check for collisions with existing stuff
+ if (OpenBSD::Vstat::vexists($fname)) {
+ push(@$colliding, $self);
+ $$problems++;
+ return;
+ }
+ $$totsize += $self->{size} if defined $self->{size};
+ my $s = OpenBSD::Vstat::add($fname, $self->{size}, \$pkgname);
+ return unless defined $s;
+ if ($s->{ro}) {
+ if ($state->{very_verbose} or ++($s->{problems}) < 4) {
+ Warn "Error: ", $s->{dev},
+ " is read-only ($fname)\n";
+ } elsif ($s->{problems} == 4) {
+ Warn "Error: ... more files can't be written to ",
+ $s->{dev}, "\n";
+ }
+ $$problems++;
+ }
+ if ($state->{forced}->{kitchensink} && $state->{not}) {
+ return;
+ }
+ if ($s->avail() < 0) {
+ if ($state->{very_verbose} or ++($s->{problems}) < 4) {
+ Warn "Error: ", $s->{dev},
+ " is not large enough ($fname)\n";
+ } elsif ($s->{problems} == 4) {
+ Warn "Error: ... more files do not fit on ",
+ $s->{dev}, "\n";
+ }
+ $$problems++;
+ }
+}
+
sub install
{
my ($self, $state) = @_;
@@ -409,6 +410,14 @@ package OpenBSD::PackingElement::Sample;
use OpenBSD::Error;
use File::Copy;
+sub validate
+{
+ my $self = shift;
+ if (!defined $self->{copyfrom}) {
+ Fatal "\@sample element does not reference a valid file\n";
+ }
+}
+
sub install
{
my ($self, $state) = @_;
@@ -416,9 +425,6 @@ sub install
my $destdir = $state->{destdir};
my $filename = $destdir.$self->fullname();
my $orig = $self->{copyfrom};
- if (!defined $orig) {
- Fatal "\@sample element does not reference a valid file\n";
- }
my $origname = $destdir.$orig->fullname();
if (-e $filename) {
if ($state->{verbose}) {
@@ -544,4 +550,44 @@ sub install
$self->mark_ldconfig_directory($state->{destdir});
}
+package OpenBSD::PackingElement::SpecialFile;
+use OpenBSD::PackageInfo;
+use OpenBSD::Error;
+
+sub validate
+{
+ my ($self, $state, $problems, $colliding, $totsize, $pkgname) = @_;
+
+ my $fname = installed_info($pkgname).$self->{name};
+ my $cname = $state->{dir}.'/'.$self->{name};
+ my $size = $self->{size};
+ if (!defined $size) {
+ $size = (stat $cname)[7];
+ }
+ if ($self->exec_on_add()) {
+ my $s2 = OpenBSD::Vstat::filestat($cname);
+ if (defined $s2 && $s2->{noexec}) {
+ Warn "Error: ", $s2->{dev}, " is noexec ($cname)\n";
+ $$problems++;
+ }
+ }
+ my $s = OpenBSD::Vstat::add($fname, $self->{size}, \$pkgname);
+ return unless defined $s;
+ if ($s->{ro}) {
+ Warn "Error: ", $s->{dev}, " is read-only ($fname)\n";
+ $$problems++;
+ }
+ if ($s->{noexec} && $self->exec_on_delete()) {
+ Warn "Error: ", $s->{dev}, " is noexec ($fname)\n";
+ $$problems++;
+ }
+ if ($state->{forced}->{kitchensink} && $state->{not}) {
+ return;
+ }
+ if ($s->avail() < 0) {
+ Warn "Error: ", $s->{dev}, " is not large enough ($fname)\n";
+ $$problems++;
+ }
+}
+
1;