Mailinglist Archive: opensuse-commit (1082 mails)

< Previous Next >
commit apparmor-provider-notifications
  • From: root@xxxxxxxxxxxxxxx (h_root)
  • Date: Fri, 04 May 2007 00:26:46 +0200
  • Message-id: <20070503222647.1AD5F6780DE@xxxxxxxxxxxxxxx>

Hello community,

here is the log from the commit of package apparmor-provider-notifications
checked in at Fri May 4 00:26:46 CEST 2007.

--------
--- apparmor-provider-notifications/apparmor-provider-notifications.changes     2007-01-12 04:54:08.000000000 +0100
+++ /mounts/work_src_done/STABLE/apparmor-provider-notifications/apparmor-provider-notifications.changes        2007-05-03 22:33:36.686203000 +0200
@@ -1,0 +2,6 @@
+Thu May  3 22:31:32 CEST 2007 - mbarringer@xxxxxxx
+
+- Fixing bug 270601.  The provider now throws an error when a 
+  notification is submitted without an email address.
+
+-------------------------------------------------------------------

Old:
----
  apparmor-provider-notifications-1.0.0.tar.gz

New:
----
  apparmor-provider-notifications-1.0.1.tar.gz

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ apparmor-provider-notifications.spec ++++++
--- /var/tmp/diff_new_pack.o26626/_old  2007-05-04 00:26:25.000000000 +0200
+++ /var/tmp/diff_new_pack.o26626/_new  2007-05-04 00:26:25.000000000 +0200
@@ -1,5 +1,5 @@
 #
-# spec file for package apparmor-provider-notifications (Version 1.0.0)
+# spec file for package apparmor-provider-notifications (Version 1.0.1)
 #
 # Copyright (c) 2007 SUSE LINUX Products GmbH, Nuernberg, Germany.
 # This file and all modifications and additions to the pristine
@@ -16,7 +16,7 @@
 %define novellprefix /usr
 %define novelllocalstatedir /var/lib/openwbem
 %define novellsysconfdir /etc/openwbem
-Version:        1.0.0
+Version:        1.0.1
 Release:        1
 License:        GNU General Public License (GPL)
 Source0:        %{name}-%{version}.tar.gz
@@ -75,6 +75,9 @@
 %dir %{novellprefix}/share/mof/%{name}
 %{novellprefix}/share/mof/%{name}/*
 
-%changelog -n apparmor-provider-notifications
+%changelog
+* Thu May 03 2007 - mbarringer@xxxxxxx
+- Fixing bug 270601.  The provider now throws an error when a
+  notification is submitted without an email address.
 * Fri Jan 12 2007 - mbarringer@xxxxxxx
 - Initial checkin of version 1.0.0 (fate #300904)

++++++ apparmor-provider-notifications-1.0.0.tar.gz -> apparmor-provider-notifications-1.0.1.tar.gz ++++++
++++ 17384 lines of diff (skipped)
++++    retrying with extended exclude list
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore --exclude Makefile.in --exclude configure --exclude config.guess --exclude '*.pot' --exclude mkinstalldirs --exclude aclocal.m4 --exclude config.sub --exclude depcomp --exclude install-sh --exclude ltmain.sh old/apparmor-provider-notifications-1.0.0/apparmor-provider-notifications.spec new/apparmor-provider-notifications-1.0.1/apparmor-provider-notifications.spec
--- old/apparmor-provider-notifications-1.0.0/apparmor-provider-notifications.spec      2007-01-12 00:50:45.000000000 +0100
+++ new/apparmor-provider-notifications-1.0.1/apparmor-provider-notifications.spec      2007-05-03 22:02:01.000000000 +0200
@@ -9,16 +9,16 @@
 #
 
 Name:          apparmor-provider-notifications
-BuildRequires: curl gcc-c++ limal-apparmor-notification-devel limal-devtools openwbem-devel
+BuildRequires: curl gcc-c++ limal-apparmor-notifications-devel limal-devtools openwbem-devel
 URL:           http://forge.novell.com/modules/xfmod/project/?apparmor
 PreReq:                /usr/bin/loadmof.sh, coreutils
 %define novellprefix /usr
 %define novelllocalstatedir /var/lib/openwbem
 %define novellsysconfdir /etc/openwbem
-Version:       1.0.0
+Version:       1.0.1
 License:       GPL
 Release:       0
-Source0:       %{name}-%{version}.tar.bz2
+Source0:       %{name}-%{version}.tar.gz
 Source1:       %{name}-mof.tar.bz2
 BuildRoot:     %{_tmppath}/%{name}-%{version}-build
 Group:         System/Management
@@ -51,7 +51,7 @@
 %post
 %run_ldconfig
 MOFDIR=%{novellprefix}/share/mof/%{name}
-loadmof.sh -v -n root/cimv2 apparmor-provider-notifications.mof
+loadmof.sh -v -n root/cimv2 $MOFDIR/apparmor-provider-notifications.mof
 
 %preun
 if [ "x$1" = "x0" ]; then
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore --exclude Makefile.in --exclude configure --exclude config.guess --exclude '*.pot' --exclude mkinstalldirs --exclude aclocal.m4 --exclude config.sub --exclude depcomp --exclude install-sh --exclude ltmain.sh old/apparmor-provider-notifications-1.0.0/config.h.in new/apparmor-provider-notifications-1.0.1/config.h.in
--- old/apparmor-provider-notifications-1.0.0/config.h.in       2007-01-12 00:20:50.000000000 +0100
+++ new/apparmor-provider-notifications-1.0.1/config.h.in       2007-05-03 22:02:18.000000000 +0200
@@ -221,7 +221,7 @@
 
 /* If using the C implementation of alloca, define if you know the
    direction of stack growth for your system; otherwise it will be
-   automatically deduced at run-time.
+   automatically deduced at runtime.
        STACK_DIRECTION > 0 => grows toward higher addresses
        STACK_DIRECTION < 0 => grows toward lower addresses
        STACK_DIRECTION = 0 => direction of growth unknown */
@@ -266,7 +266,7 @@
 /* Define to rpl_realloc if the replacement function should be used. */
 #undef realloc
 
-/* Define to `unsigned' if <sys/types.h> does not define. */
+/* Define to `unsigned int' if <sys/types.h> does not define. */
 #undef size_t
 
 /* Define to `int' if <sys/types.h> doesn't define. */
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore --exclude Makefile.in --exclude configure --exclude config.guess --exclude '*.pot' --exclude mkinstalldirs --exclude aclocal.m4 --exclude config.sub --exclude depcomp --exclude install-sh --exclude ltmain.sh old/apparmor-provider-notifications-1.0.0/configure.in new/apparmor-provider-notifications-1.0.1/configure.in
--- old/apparmor-provider-notifications-1.0.0/configure.in      2007-01-12 00:20:50.000000000 +0100
+++ new/apparmor-provider-notifications-1.0.1/configure.in      2007-05-03 22:01:51.000000000 +0200
@@ -1,4 +1,4 @@
-AC_INIT(apparmor-provider-notifications, 1.0.0, mbarringer@xxxxxxx)
+AC_INIT(apparmor-provider-notifications, 1.0.1, mbarringer@xxxxxxx)
 AC_CONFIG_SRCDIR(src)
 AM_INIT_AUTOMAKE
 AC_CONFIG_HEADER([config.h])
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore --exclude Makefile.in --exclude configure --exclude config.guess --exclude '*.pot' --exclude mkinstalldirs --exclude aclocal.m4 --exclude config.sub --exclude depcomp --exclude install-sh --exclude ltmain.sh old/apparmor-provider-notifications-1.0.0/src/AppArmorNotificationSettingDataProvider.cpp new/apparmor-provider-notifications-1.0.1/src/AppArmorNotificationSettingDataProvider.cpp
--- old/apparmor-provider-notifications-1.0.0/src/AppArmorNotificationSettingDataProvider.cpp   2007-01-12 00:20:50.000000000 +0100
+++ new/apparmor-provider-notifications-1.0.1/src/AppArmorNotificationSettingDataProvider.cpp   2007-05-03 22:00:42.000000000 +0200
@@ -342,6 +342,9 @@
                                        TerseLevel.get(tmpInt);
                                        notifyCfg["terse_level"] = String(tmpInt).c_str();
                                        TerseEmail.get(tmpStr);
+                                       if (tmpStr.empty())
+                                               OW_THROWCIMMSG(CIMException::INVALID_PARAMETER,
+                                               "Cannot allow a notification to be set up without an email address");
                                        notifyCfg["terse_email"] = tmpStr.c_str();
                                        notifyCfg["terse_unknown"] = "0";
                                        TerseUnknown.get(tmpBool);
@@ -362,6 +365,9 @@
                                        SummaryLevel.get(tmpInt);
                                        notifyCfg["summary_level"] = String(tmpInt).c_str();
                                        SummaryEmail.get(tmpStr);
+                                       if (tmpStr.empty())
+                                               OW_THROWCIMMSG(CIMException::INVALID_PARAMETER,
+                                               "Cannot allow a notification to be set up without an email address");
                                        notifyCfg["summary_email"] = tmpStr.c_str();
                                        notifyCfg["summary_unknown"] = "0";
                                        SummaryUnknown.get(tmpBool);
@@ -382,6 +388,9 @@
                                        VerboseLevel.get(tmpInt);
                                        notifyCfg["verbose_level"] = String(tmpInt).c_str();
                                        VerboseEmail.get(tmpStr);
+                                       if (tmpStr.empty())
+                                               OW_THROWCIMMSG(CIMException::INVALID_PARAMETER,
+                                               "Cannot allow a notification to be set up without an email address");
                                        notifyCfg["verbose_email"] = tmpStr.c_str();
                                        notifyCfg["verbose_unknown"] = "0";
                                        VerboseUnknown.get(tmpBool);


++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++



Remember to have fun...

---------------------------------------------------------------------
To unsubscribe, e-mail: opensuse-commit+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-commit+help@xxxxxxxxxxxx

< Previous Next >
This Thread
  • No further messages