StepConfigure - new step responsible for the InstallerContext
configuration.

Previously, AppInstaller included the general logic and some setup of
InstallerContext. However, InstallerContext should be only used within steps
(AppInstaller) should not mix the data and the logic.

Hence, Almost all context setup was moved to separate, new step - Configure

Change-Id: I508d2383bd6fca66326aa18df1bedb6f52fae3a6

removal of not needed method call.

PkgmgrSingal::SendProgress method will just return false, if
state != Statue::Started.
This condition is always true in the beginning of Run() method.
Hence, this method will not perform anything.

Change-Id: I2ff46b156c931e9259fb434179a2a687a7ba93fa

Merge "Adding Samsung License to files modified by Samsung engineers Adding Samsung engineers to AUTHORS files" into tizen

Introducing "precheck" method for each step.

Precheck method should verify, if all necessary input data has been provided.
It returns OK on success (all necessary data has been provided to perform
"process" method).

Appinstaller, for each step, calls these methods in following order:
for(step: steps) {
  ...
  if (step->precheck() != Step::Status::OK)
    break;
  step->process();
  ...
}

TODO:
* step_parse for WGT and TPK should introduce it
* step_create_symbolic_link for WGT and TPK should introduce it

Change-Id: Ibd506603bfb5c152723bae13e56301b517e94b2b

Adding Samsung License to files modified by Samsung engineers
Adding Samsung engineers to AUTHORS files

Change-Id: I58001e73e7d9605cf92ae6a50e161bb18a8616f4

Extract MoveDir() to utils

Change-Id: Iae4d60d42b9c90003de4ed23070c9e34d358991d

Quickfix context->application_path

This field was set in context installer constructor where pkgid is
not yet available in installation case.

Change-Id: Ida6f97c763a2f3eec780d0c73fef08562de7547b

Context paths types changed to boost::filesystem::path

Change-Id: Ic82ebf53ecbae4029c39332e6fd057719d93a78d

[Manifest parser] Remove manifest parser

Remove manifest-parser due to moving it to
separate repository.

Change-Id: I99dc8f7f363f0fed98a97a172643914b159aed04

Revert "Xml manifest path in InstallerContext"

This commit needs to be reverted as it is removing setting up
variable which is later used in generating xml. Therefore
widgets do not install as crash occurs when calling function
on empty variable

Change-Id: Ic91a94851e00c35e4fa5bcd77c90b4dc556b7562
Signed-off-by: Wojciech Kosowicz <w.kosowicz@samsung.com>

Fix rollback for installation mode

There is an issue when we rollback an operation after appling
SMACK labels to widgets files.

In rollback security context is revoked before removing files
of widget. This will cause failure on rollback because
installer will try to remove files that it has no access to.

Bug-Tizen: TC-2484

Change-Id: Ic171732de546a62dd451be0ae39abdae18e76628

Xml manifest path in InstallerContext

This commit changes xml_path type to fs::type.
It also removes unnecessary setting from step_generate_xml

Change-Id: I2bc99da823959a4968802ca47367b6b1f2fca57c
Signed-off-by: Pawel Sikorski <p.sikorski@samsung.com>

Remove hardcoded library path from PC files

BUG-Tizen:https://bugs.tizen.org/jira/browse/TC-2492

Change-Id: I5e69865c214b63401658d4602af080bb9b5bb190
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

[Manifest parser] Create devel package, add new rpm

Create manifest-parser devel package,
add manifest-parser.pc pkgconfig file.
This also creates new rpm package for manifest parser
from, which app-installers should be dependant.

Change-Id: I70a744d138bce12eccb90705f80a2d1665cf91f4

[Manifest parser] Separate filling manifest_x into few steps

Separate filling manifest_x structure into steps:
- filling widget info
- filling application info
- filling icon paths
- filling privileges
- filling app control

Change-Id: Ie4ea566bb797d2b7af196899663ff74342a1b493

Remove assert checking for existance of widget package file

Remove assert, which checks if widget package file exsits
and replace it with checks from boost lib for file existence and
return error on failure.

Change-Id: I75dd117d85bef0fe1313da8a2510fe5bd6673dc7

[Unit tests] Add unit tests for signature

3/3 tests PASS

Change-Id: I45e1e41b8e94ac171e75ad4e047cbd0d5379bb3b

Allowing multiple instances of PkgmgrSignal.

This commit unifies states between multiple instances of
PkgmgrSignal objects (using static variable).

Change-Id: I698c509c60533534c49ba29f537aca0e56ae63a9

Removal of request_type property from InstallerContext.

It is not used anywhere.

Change-Id: I6efd669e8b1827bb37659db0f13543029283aa31

Introducing PkgMgrInterface to separate app-installer from
external library.

TODO:
* PkgMgrSignal should be refactor. Possibly, it could inherit from
  PkgMgrInterface

Change-Id: I3f934674aec54c69073a122fcb4b71e16a94565f

[Signal] Sending progess to pkgmgr

Simple approach, calculate progress:

  progress = numberOfStepExecuted * 100 / totalNumberOfSteps

No point to make it harder by now, IMO.

Change-Id: If20174e3756e1c087d8cac8fd4704bb82c1d5d13

[Signal] Send 'error' code to pkgcmd

All logic calling SignalPkgmgr class was moved to
AppInstaller class.

Change-Id: Ic99423abab4eaf5f239a27104b653e1dd12a8ad2
Signed-off-by: Pawel Sikorski <p.sikorski@samsung.com>

Fix for not closed app-control element during xml generation

Commit https://review.tizen.org/gerrit/#/c/36834/ introduced
bug related to not closing app-control element and as a result
privilege element is inserted into ui-application element and
manifest don't pass validation.

Change-Id: If6e50fe196ffe783d307d80fb48821eb87a2da94

Bring to common xml generation code

Make template function for generating common xml code
for uiapplication_x and serviceapplication_x

Make template funct

Change-Id: Iface15d6aed7055da0b42ba04ca374c8964cb880

Introducing "Property" for ContextInstaller.

Currently, ContextInstaller contains attributes (declared as private).
And each attribute has its setter and getter defined.

Property hides declaration of these setters/getters.

TODO:
* Pi (PkgmgrSignal)

Change-Id: Iba6e8a796b54468798b35e708a8ecee03ec53988

Unification of manfiest parser

This change includes changes according to architecture
planned in https://bugs.tizen.org/jira/browse/TC-2472
Further changes will be made later in next commits.

Change-Id: I9b3f2c9d340994b0248ba002d06effc70f30bffd

add manifest for tpk backend

Change-Id: I5d632d00f86619111f222923c82f1d39da735621
Signed-off-by: Baptiste DURAND <baptiste.durand@gmail.com>

Rename:
* StepSecurity -> StepRegisterSecurity
* StepSignature -> StepCheckSignature
* StepRemove -> StepRemoveFiles
* StepSymbolicLink -> StepCreateSymbolicLink
* StepRecord -> StepRegisterApplication
* StepUnregister -> StepUnregisterApplication

Most of the steps contain verb and noun in their names.
Hence, commit unifies all steps to follow this rule.

Change-Id: I59114dc41385b16e4661e23cd38fcb65637289a3

[Unittest] service_handler_unittests

This commit removes checks on GetManifestData()
as it ws used for multithreading purpose in crosswalk.

Change-Id: I2c3a19f834bb2af02f9319008ed813d427b2460c

[Parser] Add tizen:service handler

Spec: #1730

https://source.tizen.org/sites/default/files/page/tizen-2.3-wrt-core-spec.pdf

ValidateTizenApplicationId() was extracted due to use in multiple handlers.

Change-Id: I2c86d6a8f3e73709c57447781f96d62cb3cb8029

Available Storage Check

Solution calculates space needed for files.
This will ignore space for directories/icons/manifests but
in overall will prevent unneccesary unzipping the archive
in most cases.

Change-Id: Ib8e81c4cb56e771347a128021ece53e4cf2c9fce

[Copy] Move files instead of copy if possible

If move is not possible then fall back to copy/remove.

Change-Id: Icf2dbbf63b772ae6eccb4682a4295b397ca91548

Support full spec of tpk manifest xml

With this commit, the tpk backend parses the full spec of the
tizen-manifest.xml.

* Multiple children having same name are parsed well.
* <author>, <service-application> are parsed.
* More attributes which are in the spec but not parsed are
  supported.

Change-Id: I67689841125bb97048fa10accddab8e7e3839984
Singed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Fix UTF8CharLength

This function is wrong because of char's 'signness' is
implementation defined.

C++ standard doesn't define char as unsigned char.
Basically,
  char != signed char
  char != unsigned char

This was ok for arm architeture but not for x86.
This overlook caused calculated length of character
to be wrong.

Change-Id: Ifd826ae38098e78da8de858fd5720a6a3dc46a38

Merge "Allow optional elements in tpk manifest xml" into tizen

Move/rename widget-manifest-parser

Simplify naming and remove underscores in directory name.

Change-Id: I3df8b3212c70e6456e8cc47c0e3f263c1ebdc514

[Security] Fix passing path in StepRevokeSecurity::undo()

RegisterSecurityContextForApps() takes path to the root of
application (where the index.html is located).

Same as in: https://review.tizen.org/gerrit/#/c/36757/

Change-Id: Id46a569e4d140951edeecf7e7fba1c377c2b2daf

Allow optional elements in tpk manifest xml

Some xml elements are optional, thus they can be omitted in the
tizen-manifest.xml.
This patch allows those elements to be omitted.
In addition, null checking routine in common/step/step_generate_xml.cc
is added, because if there is no description in tizen-manifest.xml,
manifest_x->description will be nullptr.

Change-Id: Ie37236b472011c9d516840cc9cda86b777a6a474
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Fix parser handlers

In current execution flow, there is call of Parse() function
for each handler. Handler should be prepared for situation that
there is no value it expects.

Change-Id: Ib17ed3e4bbb5600397c9bf23cf140d0d064b8d29

Fix manfiest_util parsing localized text nodes

manifest_util was was adding bidi control characters in front and
in the back twice, when dir attribute was specified.

Manifest util test results:
12/12 tests PASS

Change-Id: Ic83a766508f871607849c298d335e813b4484db7

[Unitttests] string_utils_unittest

This commit adds unit tests for 'src/utils/string_util.*'

Change-Id: I237ebe67f7f761afb5f6dfa5d6d648206e8f7d13

Fix tpk parser bug

Fixed condition of judging children count of an element

Change-Id: I9547294a2c1b1018a0bed87e4eab656a61ed2e2e
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Perform StepRevokeSecurity after StepRemove

Revoking security configuration causes StepRemove to fail.
Application files are labeled with per-app Smack label.
Access to that label is part of security configuration which shouldn't
be revoked before files are uninstalled.

Change-Id: Ib41ca3b04324f307954c64fa93cdefaf9da0325b
Signed-off-by: Rafal Krypa <r.krypa@samsung.com>

Add missing unit tests for manifest util

10/12 tests PASSES

GetNodeText tests fails due to double addition
of bidi control characters

Change-Id: I766827bd7b61da469fd38376bdee1c6715034e2f

Remove Manifest::Type from parser

Remove Manifest::Type, as this parser was supposed to be
only for config.xml from *.wgt applications.

Change-Id: Ie465f45c40d2c59492e3f9503132af8e47a4ccd0

Fix the path of each application passed to security manager

Wrong path of each application was passed to the security manager. It
causes installation failure.
This commit fixes the installed application path.

Change-Id: I0b95e528ef2f4c41091b4e60bb9f42e1dce02333
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>
Signed-Off-By: Sangyoon Jang <s89.jang@samsung.com>

Replace tpk manifest parser to XmlParser

Old manifest parser is replaced with the new general-purpose XmlParser.
With the XmlParser, the element hierarchy is fully preserved.
In addition, all exceptions are removed in this commit.

Change-Id: Id03384dc0b3c661bd876966bfbb3883b631cc68d
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Add 'app-control' manifest handler

Change-Id: Ib06a69de03eddbac768c8ada921dfed60b33e006

Fix build break by applying new file_util namespace

With the commit f2251075, src/common/utils.h is moved to
src/utils/file_util.h, and the corresponding namespace is changed too.
But the commit did not fix the tpk source, and it causes build break.
This commit fixed the build break, and make the tpk installer work as
well.

Change-Id: Ibd55bdcb7dfcc460233c8f0bfcc7de5cdf8e236c
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Fix for passing compile flags correctly in cmake

This fixes ApplyPkgConfig.cmake module.
Flags should be passed to target if there are present
in pkg-config. Skipping them, as it was done, may
lead to hidden errors in code.

Change-Id: I41c25b04e329d6379efb5736eea1973ce68d674c

Move file_util.cc to utils

Change-Id: Ida4e0c8c922549165f318b7ca3292d1a4fd381ba

Remove version check in ApplicationData

Remove getting manifest version attribute error code, as
according to w3c specification version attribute is optional
and can be any string.

Links to spec:
http://www.w3.org/TR/widgets/#the-version-attribute and
http://www.w3.org/TR/widgets/#version-attribute

Change-Id: I5d8192a34721d8e7c9ac08af70e98af9fd011dda

Fix issues with icon

Fix generation of xml manifest when there is no icon defined in
config.xml. Fix checking if icon exists during fill manifest_x
structure.

Change-Id: I1c1700fc7918eb16a6c94fde77bf5b33818208d5

Fix string buffer overwrite

std::string uses static buffer for empty string values. When
std::transform iterates over initialized std::string elements
and copies their changed values to empty std::string by using
its random access iterator, the buffer which is used by
std::string for empty strings is overriden, what later makes
all strings, that are supposed to be empty, not empty.

Change-Id: I5acb9e038febabbf4b1e1d56d736f75dc227c0af

Fix string_utils.cc

This commit fixes mistake in EqualsUTF8Char function and
removes not needed parameter from CollapseWhitespaceUTF8

Change-Id: Idee35b3df00da7dd4938d0151bbd6c9a3ca6132c

Merge "Fix generating app-control xml node in platform manifest" into tizen

Fix generating app-control xml node in platform manifest

Change-Id: Ib6d6c2965f8b63e27c9455915c0f859fa200112a

Stylecheck fix

Change-Id: I9efb79ec07d84a6366c6f75307218d8a195ce5cf

Remove compiler's warnings

Change-Id: I86fe2db1d6a947d9d8f5bcc1dad82698821024b4

[Unit tests] Add unit test target and few unit tests.

This patch adds tests target which includes unit tests executables:
- manifest_handler_unittest (3/3 tests PASS)
- manifest_unittest (2/2 tests PASS)
- manifest_util_unittest(2/2 tests PASS)
- values_unittest (20/20 tests PASS)
- widget_manifest_parser_unittest (2/2 tests PASS)

Tests are installed in /usr/bin/app-installers-ut directory.
Resources for tests are installed in /usr/share/app-installers-ut
directory.

Change-Id: I4883a1d4d6499cad3cde5e0b635467cfa042c513

[Widget manifest parser] Remove C API from parser

Change C API functions into class containing ParseManifest,
GetShortName and GetRequiredAPIVersion methods, which allow
removal of additional data structure used for containing
config.xml data which are not part of manifest_x

Change-Id: I56ea88500de2a5a9838326f7c432276c02d18095

Add TPK backend

Native tpk backend implementation: First version.
Only simple tizen-manifest.xml format can be parsed for now.

Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>
Change-Id: I95037fa6b08639c6dd008d0ffb796cfd375c8060

Fix bug in traversing pkgmgr-info lists again

Same issue to a5cb75ca.
This commit prevents segfault from accessing null pointer.

Change-Id: I0aac62923a5948f067c54b05b3edb88d39736c0e
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Fix bug in traversing pkgmgr-info lists

When the list is null, the LISTADD macro raises segfault.
Added null check routine on the macro in app-installer.

Basically this bug must be fixed in the pkgmgr-info, not here.

Change-Id: I8aa557c59c5771a817899e8fdb21349fd88ba511
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Fix build error in wgt

ReleaseData() method returns nothing, but the current code tries to
get a return value from it. This patch removes the return value
cheching.
If the return value checking is really needed, the definition of
the method must be modified.

Change-Id: I22975cb10c64cef114a4ece58aafbec5257ec373
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

[Signature] Fix double log

In case of invalid signature there was double log:
"Signature * is INVALID
 Signature * is OK"
This fix this bug.

Change-Id: Ief1f119f8317c36b3ed0b230081a633431c01e69

[Widget-manifest-parser] Remove LocalManifestData

Clean up in widget_manifest_parser.cc file, which
includes removal of LocalStorage and LocalManifestData.

Change-Id: Ie09349504fe014ac807253f4de87c19dc98e2522

[Signature] Add XMLSEC_NO_SIZE_T flag to cmake

This flag is needed, because otherwise _xmlSecDSigCtx structure
will use size_t in app-installers and unsigned int in xmlsec1 for sizes
As a result structure size in function from xmlsec1 will differ from
size in app-installers

Change-Id: I241d04d95ef6297f982bd95d6220b1782e6f7981

Remove label element under manifest

'manifest' element doesn't have 'label' child element. This commit
removes unnecessary label element.

Change-Id: I6d8a68ea702c2ea30ec0c5ed316873e503e632be

Add error logs on revoke_security step

In addition, an error message is printed as an error, not a debug
message.

Change-Id: I1adb4fe8615a33652d3b00875adad52a6c9627d4
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Fix wrong manifest xml file name

The manifest file name is wrong. Desired xml filename is
'org.tizen.hello.xml', but the current implementation removes last
extension and put '.xml' at the end.
This commit fixes the name of the generated manifest xml file.
For wgt packages, '*.wgt.xml' is present, but that's not the issue.

Change-Id: I57e2506898f17c0853fa42f3172225b69cd9819c
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Fix bugs in common steps

* Fixed generating xml routine, that each privilege have its value
  as a data, not a 'name' attribute.
* Changed the iteration routine, from the first item of the privilege
  list.
* Excluded routine removing xml file, from the 'undo' phase of the
  step_record.

Change-Id: Ied4ff4a48ab0a62f93fbb4fac40faa00952eaefe
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Refactor name unpack_directory()

The method unpack_directory() just returns the unpack_directory string,
    but its name implies for class users to "Unpack the directory".
    This commit changes the name from unpack_directory to
    unpacked_dir_path, of the getter, setter, and private variables.

Change-Id: I7259bbb3c8630d7464a98707162d955de45c9432

Give the package type as a AppInstaller object

To send a start/end/progress DBUS signal, client needs to know the
package type as a string. This patch adds an argument package_type on
the AppInstaller constructor, and stores it into the ContextInstaller.

Change-Id: I18a7770dbb706a6ab72cce6010d348fda4d47137

Add parser's handler: appwidget

Change-Id: I3d4009960593e319abc9034e5c6e27b34e064e25

Add parser's handlers: metadata, navigation, splash screen

Change-Id: Idad70b51377d8020a4cc71ae44e382a869ff4a3c

Add setting parser's handler

Change-Id: I2063f0aa5046f5981b2abbb41576d3d1150e9009

Add IME parser's handlers

Change-Id: I8b316d4ce04f05397e05903cd3a921cd05013637

Escape encoded character in URLs in manifest

Change-Id: I30667246ca44bd66ca80953ac9f54555a2564c34

DISALLOW_COPY_AND_ASSIGN(AppInstaller)

Change-Id: I641fa3bf29ec3147e54430259eb6a6d87596c202

Logging unification

Because of existing idea of tagging step in logs,
this commit introduces one more macro:

SCOPE_LOG_TAG(TAG)

Defining this macro in given scope will change log tag of that
scope.

Example:

class StepXYZ : public Step {
  void func() {
    LOG(DEBUG) << "Status is ...";
  }

  SCOPE_LOG_TAG(XYZ)
};

Every log from class StepXYZ will be place with "XYZ" tag.

Change-Id: I8a8e192b0a746dae19a961e238a798b9b2c0c4b9

Remove utf16 handling

This code is not needed.
There is no need for utf16 error codes of parser.

value.h make no longer assumtions on string
encoding - there is no check.

Change-Id: I2d22b8683b27a659c237c0fd1bf167e37c15db70

[Widget-manifest-parser] Added parser's base

Change-Id: Ic7325a08d9ed1359a6e6be77e0d62b3dbe046e7f

[Widget-manifest-parser] Add manifest class and manifest constants

This change adds Manifest class, which is internal representation
of manifest file in widget-manifest-parser.

Change-Id: Icf9a53398fe3941cb19e36d9bacad721ffa0b057

Move logging and macros to utils

Just moves code. I will think about appling this.

Change-Id: I6e6e21a9144f548697acc0f959e7aa2338831121

Merge "Add utils library." into tizen

Send pkgmgr signal always

Zip or config.xml can be malformed.
This fixes handling signalling in those cases.

Code which handles pkgmgr signal was extracted into
 - src/common/pkgmgr_signal.*

Change-Id: I46b96f6ba8cd2c41110f2fefbceb6dc1b2969216

Merge "Add url decoding for 'uri' attribute of signatures" into tizen

Add url decoding for 'uri' attribute of signatures

The value of URI attribute of signature files can be encoded uri.
But, SignatureValidator can't recongize encoded 'uri'.
Add url decoding routine to SignatureParser.

Change-Id: Ifec7b04b2a591bddca15fbe3cdafa90d2a0964c5

Add utils library.

It is supposed to contain common utilities, which can be used by
other parts of code. This commit contains utf_converter code,
which allows convertions between std::u16string and std::string.
There is also DictionaryValue class added, which is needed for
saving nodes of widget manifest.

Change-Id: Ibe4265e6dc3a7f78637d0103222d87d16b70bc2a

Remove double installer creation

Change-Id: Id8842d3dcb471a00437f3d620229a6c45a97989f

Manifest generation fix

xmlTextWriterEndDocument closes all not closed elements (nodes).
Therefore it faked that manifest is ok.
Some elements were not closed properly.

Change-Id: I0b4e1a0051c25dff10a67a7ac8809295d44d5c5b

Fix Name field in case of widget label

If the short name is empty use the long name instead

Change-Id: Id7f6bd3fa54a9801c0f93e510ae6f723241185e4
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Fix Description field in Tizen Manifest

This permits to avoid to have this

<label>(null)</label> instead of <label></label> in the manifest

Change-Id: Iddaedf33940358cd52590487f6a7c6febda809c1
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Fix Application type in Generate XML Manifest step.

The application should depends of the original type set in the original manifest.

Now, use the one provided by step parser

Change-Id: I6aa34cf1dc2ebc9733f6239125aa12f441de47c0
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Add new specific step to create symbolic link for WebApp

Remove this operation from generate xml step, create a new one

Add a spec file option for set the WRT binary path used for symbolic link

Change-Id: Ife93a10b830fd093dc9c02184dd2490ce99d67a0
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Add manifest to wgt-backend package.

This is needed to assign the "User" smack label to wgt-backend

Change-Id: Icea7b182c89a7e8ff631f6f6ce9f188d180c8fd7
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

StepSecurity

Adds security manager API calls to apply correct security context
during installation and uninstallation process.

Change-Id: Ieceb1126edfe65f1f1f438e9c4dea7907e9f3e27

Step class design issues

 - lack of virtual destructor
   It is obligatory in classes that are extended.
   This case may lead to memory leak or undefined behaviour.

 - unnecessary argument passing to each function,
   We have 'class' to keep information within.

 - memory management in AppInstaller class of allocated Steps,
   Most reliable way is to use smart pointers.
   Use new/delete only if it is really neccesary.

 - fixes errorcodes to some understandable ones,
   Any function should not return int that cannot be interpreted
   what it actually represents.

 - catching int,
   There was some mistake (?). Use std::exception derived classes in
   exception handling if needed.

Change-Id: Iba8ea02dde503ec1b2f043ed5965a537d6a58f51

Fix error Management

If Undo method fails the Error's signal is not sent to pkgmgr-server
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>
Change-Id: Ib867d1a0f488787b2408f914e66ab561d225d65a