From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from pigeon.gentoo.org ([208.92.234.80] helo=lists.gentoo.org)
	by finch.gentoo.org with esmtp (Exim 4.60)
	(envelope-from <gentoo-commits+bounces-393178-garchives=archives.gentoo.org@lists.gentoo.org>)
	id 1RFsrV-0005MU-9u
	for garchives@archives.gentoo.org; Mon, 17 Oct 2011 19:28:37 +0000
Received: from pigeon.gentoo.org (localhost [127.0.0.1])
	by pigeon.gentoo.org (Postfix) with SMTP id 306A321C0C2;
	Mon, 17 Oct 2011 19:28:30 +0000 (UTC)
Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183])
	by pigeon.gentoo.org (Postfix) with ESMTP id CD2D621C0C2
	for <gentoo-commits@lists.gentoo.org>; Mon, 17 Oct 2011 19:28:29 +0000 (UTC)
Received: from pelican.gentoo.org (unknown [66.219.59.40])
	(using TLSv1 with cipher AECDH-AES256-SHA (256/256 bits))
	(No client certificate requested)
	by smtp.gentoo.org (Postfix) with ESMTPS id 411AB1B4019
	for <gentoo-commits@lists.gentoo.org>; Mon, 17 Oct 2011 19:28:29 +0000 (UTC)
Received: from localhost.localdomain (localhost [127.0.0.1])
	by pelican.gentoo.org (Postfix) with ESMTP id 5F00780042
	for <gentoo-commits@lists.gentoo.org>; Mon, 17 Oct 2011 19:28:28 +0000 (UTC)
From: "Anthony G. Basile" <blueness@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Content-type: text/plain; charset=UTF-8
Reply-To: gentoo-dev@lists.gentoo.org, "Anthony G. Basile" <blueness@gentoo.org>
Message-ID: <f1097c9e8fc1624799f7a08c7e510e24903f1f7d.blueness@gentoo>
Subject: [gentoo-commits] proj/elfix:master commit in: scripts/
X-VCS-Repository: proj/elfix
X-VCS-Files: scripts/paxmodule.c scripts/revdep-pax
X-VCS-Directories: scripts/
X-VCS-Committer: blueness
X-VCS-Committer-Name: Anthony G. Basile
X-VCS-Revision: f1097c9e8fc1624799f7a08c7e510e24903f1f7d
Date: Mon, 17 Oct 2011 19:28:28 +0000 (UTC)
Precedence: bulk
List-Post: <mailto:gentoo-commits@lists.gentoo.org>
List-Help: <mailto:gentoo-commits+help@lists.gentoo.org>
List-Unsubscribe: <mailto:gentoo-commits+unsubscribe@lists.gentoo.org>
List-Subscribe: <mailto:gentoo-commits+subscribe@lists.gentoo.org>
List-Id: Gentoo Linux mail <gentoo-commits.gentoo.org>
X-BeenThere: gentoo-commits@lists.gentoo.org
Content-Transfer-Encoding: quoted-printable
X-Archives-Salt: 
X-Archives-Hash: e90a8bf80edc0ad76df0a0f15290dc10

commit:     f1097c9e8fc1624799f7a08c7e510e24903f1f7d
Author:     Anthony G. Basile <blueness <AT> gentoo <DOT> org>
AuthorDate: Mon Oct 17 19:28:22 2011 +0000
Commit:     Anthony G. Basile <blueness <AT> gentoo <DOT> org>
CommitDate: Mon Oct 17 19:28:22 2011 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=3Dproj/elfix.git;a=3D=
commit;h=3Df1097c9e

scripts/{paxmodule.c,revdep-pax}: add code to set pax flags

---
 scripts/paxmodule.c |  183 +++++++++++----------------------------------=
------
 scripts/revdep-pax  |  100 +++++++++++++++++-----------
 2 files changed, 101 insertions(+), 182 deletions(-)

diff --git a/scripts/paxmodule.c b/scripts/paxmodule.c
index eac774a..32c8768 100644
--- a/scripts/paxmodule.c
+++ b/scripts/paxmodule.c
@@ -1,6 +1,5 @@
 #include <Python.h>
=20
-#include <stdio.h> //remove when you remove printf
 #include <string.h>
=20
 #include <gelf.h>
@@ -53,14 +52,13 @@ static PyObject *
 pax_getflags(PyObject *self, PyObject *args)
 {
 	const char *f_name;
-	int fd, sts;
+	int fd;
 	Elf *elf;
=20
 	char pax_buf[BUF_SIZE];
+	uint16_t pax_flags;
=20
 	GElf_Ehdr ehdr;
-	uint16_t ei_flags;
-
 	GElf_Phdr phdr;
 	char found_pt_pax;
 	size_t i, phnum;
@@ -102,6 +100,8 @@ pax_getflags(PyObject *self, PyObject *args)
=20
=20
 	found_pt_pax =3D 0;
+	pax_flags =3D 0;
+
 	elf_getphdrnum(elf, &phnum);
 	for(i=3D0; i<phnum; ++i)
 	{
@@ -116,24 +116,25 @@ pax_getflags(PyObject *self, PyObject *args)
 		if(phdr.p_type =3D=3D PT_PAX_FLAGS)
 		{
 			found_pt_pax =3D 1;
+			pax_flags =3D phdr.p_flags;
=20
-			pax_buf[0] =3D phdr.p_flags & PF_PAGEEXEC ? 'P' :
-				phdr.p_flags & PF_NOPAGEEXEC ? 'p' : '-' ;
+			pax_buf[0] =3D pax_flags & PF_PAGEEXEC ? 'P' :
+				pax_flags & PF_NOPAGEEXEC ? 'p' : '-' ;
=20
-			pax_buf[1] =3D phdr.p_flags & PF_SEGMEXEC   ? 'S' :=20
-				phdr.p_flags & PF_NOSEGMEXEC ? 's' : '-';
+			pax_buf[1] =3D pax_flags & PF_SEGMEXEC   ? 'S' :=20
+				pax_flags & PF_NOSEGMEXEC ? 's' : '-';
=20
-			pax_buf[2] =3D phdr.p_flags & PF_MPROTECT   ? 'M' :
-				phdr.p_flags & PF_NOMPROTECT ? 'm' : '-';
+			pax_buf[2] =3D pax_flags & PF_MPROTECT   ? 'M' :
+				pax_flags & PF_NOMPROTECT ? 'm' : '-';
=20
-			pax_buf[3] =3D phdr.p_flags & PF_EMUTRAMP   ? 'E' :
-				phdr.p_flags & PF_NOEMUTRAMP ? 'e' : '-';
+			pax_buf[3] =3D pax_flags & PF_EMUTRAMP   ? 'E' :
+				pax_flags & PF_NOEMUTRAMP ? 'e' : '-';
=20
-			pax_buf[4] =3D phdr.p_flags & PF_RANDMMAP   ? 'R' :
-				phdr.p_flags & PF_NORANDMMAP ? 'r' : '-';
+			pax_buf[4] =3D pax_flags & PF_RANDMMAP   ? 'R' :
+				pax_flags & PF_NORANDMMAP ? 'r' : '-';
=20
-			pax_buf[5] =3D phdr.p_flags & PF_RANDEXEC   ? 'X' :
-				phdr.p_flags & PF_NORANDEXEC ? 'x' : '-';
+			pax_buf[5] =3D pax_flags & PF_RANDEXEC   ? 'X' :
+				pax_flags & PF_NORANDEXEC ? 'x' : '-';
 		}
 	}
=20
@@ -147,20 +148,20 @@ pax_getflags(PyObject *self, PyObject *args)
 			return NULL;
 		}
=20
-		ei_flags =3D ehdr.e_ident[EI_PAX] + (ehdr.e_ident[EI_PAX + 1] << 8);
+		pax_flags =3D ehdr.e_ident[EI_PAX] + (ehdr.e_ident[EI_PAX + 1] << 8);
=20
-  		pax_buf[0] =3D ei_flags & HF_PAX_PAGEEXEC ? 'p' : 'P';
-		pax_buf[1] =3D ei_flags & HF_PAX_SEGMEXEC ? 's' : 'S';
-		pax_buf[2] =3D ei_flags & HF_PAX_MPROTECT ? 'm' : 'M';
-		pax_buf[3] =3D ei_flags & HF_PAX_EMUTRAMP ? 'E' : 'e';
-		pax_buf[4] =3D ei_flags & HF_PAX_RANDMMAP ? 'r' : 'R';
-		pax_buf[5] =3D ei_flags & HF_PAX_RANDEXEC ? 'X' : 'x';
+  		pax_buf[0] =3D pax_flags & HF_PAX_PAGEEXEC ? 'p' : 'P';
+		pax_buf[1] =3D pax_flags & HF_PAX_SEGMEXEC ? 's' : 'S';
+		pax_buf[2] =3D pax_flags & HF_PAX_MPROTECT ? 'm' : 'M';
+		pax_buf[3] =3D pax_flags & HF_PAX_EMUTRAMP ? 'E' : 'e';
+		pax_buf[4] =3D pax_flags & HF_PAX_RANDMMAP ? 'r' : 'R';
+		pax_buf[5] =3D pax_flags & HF_PAX_RANDEXEC ? 'X' : 'x';
 	}
=20
 	elf_end(elf);
 	close(fd);
=20
-	return Py_BuildValue("s", pax_buf);
+	return Py_BuildValue("si", pax_buf, pax_flags);
 }
=20
=20
@@ -168,10 +169,11 @@ static PyObject *
 pax_setflags(PyObject *self, PyObject *args)
 {
 	const char *f_name;
-	int pax_flags;
-	int fd, sts;
+	uint16_t pax_flags;
+	int fd;
=20
 	Elf *elf;
+
 	GElf_Ehdr ehdr;
 	uint16_t ei_flags;
=20
@@ -190,13 +192,13 @@ pax_setflags(PyObject *self, PyObject *args)
 		return NULL;
 	}
=20
-	if((fd =3D open(f_name, O_RDONLY)) < 0)
+	if((fd =3D open(f_name, O_RDWR)) < 0)
 	{
 		PyErr_SetString(PaxError, "pax_setflags: open() failed");
 		return NULL;
 	}
=20
-	if((elf =3D elf_begin(fd, ELF_C_READ_MMAP, NULL)) =3D=3D NULL)
+	if((elf =3D elf_begin(fd, ELF_C_RDWR_MMAP, NULL)) =3D=3D NULL)
 	{
 		close(fd);
 		PyErr_SetString(PaxError, "pax_setflags: elf_begin() failed");
@@ -211,8 +213,6 @@ pax_setflags(PyObject *self, PyObject *args)
 		return NULL;
 	}
=20
-
-
 	if(gelf_getehdr(elf, &ehdr) !=3D &ehdr)
 	{
 		elf_end(elf);
@@ -223,54 +223,48 @@ pax_setflags(PyObject *self, PyObject *args)
=20
 	ei_flags =3D ehdr.e_ident[EI_PAX] + (ehdr.e_ident[EI_PAX + 1] << 8);
=20
+	ei_flags &=3D ~HF_PAX_PAGEEXEC;
+	ei_flags &=3D ~HF_PAX_SEGMEXEC;
+	ei_flags &=3D ~HF_PAX_MPROTECT;
+	ei_flags |=3D HF_PAX_EMUTRAMP;
+	ei_flags &=3D ~HF_PAX_RANDMMAP;
+	ei_flags |=3D HF_PAX_RANDEXEC;
+
 	//PAGEEXEC
 	if(pax_flags & PF_PAGEEXEC)
 		ei_flags &=3D ~HF_PAX_PAGEEXEC;
 	if(pax_flags & PF_NOPAGEEXEC)
 		ei_flags |=3D HF_PAX_PAGEEXEC;
-	if((pax_flags & PF_PAGEEXEC) && (pax_flags & PF_NOPAGEEXEC))
-		ei_flags &=3D ~HF_PAX_PAGEEXEC;
=20
 	//SEGMEXEC
 	if(pax_flags & PF_SEGMEXEC)
 		ei_flags &=3D ~HF_PAX_SEGMEXEC;
 	if(pax_flags & PF_NOSEGMEXEC)
 		ei_flags |=3D HF_PAX_SEGMEXEC;
-	if((pax_flags & PF_SEGMEXEC) && (pax_flags & PF_NOSEGMEXEC))
-		ei_flags &=3D ~HF_PAX_SEGMEXEC;
=20
 	//MPROTECT
 	if(pax_flags & PF_MPROTECT)
 		ei_flags &=3D ~HF_PAX_MPROTECT;
 	if(pax_flags & PF_NOMPROTECT)
 		ei_flags |=3D HF_PAX_MPROTECT;
-	if((pax_flags & PF_MPROTECT) && (pax_flags & PF_NOMPROTECT))
-		ei_flags &=3D ~HF_PAX_MPROTECT;
=20
 	//EMUTRAMP
 	if(pax_flags & PF_EMUTRAMP)
 		ei_flags |=3D HF_PAX_EMUTRAMP;
 	if(pax_flags & PF_NOEMUTRAMP)
 		ei_flags &=3D ~HF_PAX_EMUTRAMP;
-	if((pax_flags & PF_EMUTRAMP) && (pax_flags & PF_NOEMUTRAMP))
-		ei_flags &=3D ~HF_PAX_EMUTRAMP;
=20
 	//RANDMMAP
 	if(pax_flags & PF_RANDMMAP)
 		ei_flags &=3D ~HF_PAX_RANDMMAP;
 	if(pax_flags & PF_NORANDMMAP)
 		ei_flags |=3D HF_PAX_RANDMMAP;
-	if((pax_flags & PF_RANDMMAP) && (pax_flags & PF_NORANDMMAP))
-		ei_flags &=3D ~HF_PAX_RANDMMAP;
=20
 	//RANDEXEC
 	if(pax_flags & PF_RANDEXEC)
 		ei_flags |=3D HF_PAX_RANDEXEC;
 	if(pax_flags & PF_NORANDEXEC)
 		ei_flags &=3D ~HF_PAX_RANDEXEC;
-	if((pax_flags & PF_RANDEXEC) && (pax_flags & PF_NORANDEXEC))
-		ei_flags |=3D HF_PAX_RANDEXEC;
-
=20
 	ehdr.e_ident[EI_PAX] =3D (uint8_t)ei_flags  ;
 	ehdr.e_ident[EI_PAX + 1] =3D (uint8_t)(ei_flags >> 8) ;
@@ -283,6 +277,7 @@ pax_setflags(PyObject *self, PyObject *args)
 		return NULL;
 	}
=20
+
 	elf_getphdrnum(elf, &phnum);
 	for(i=3D0; i<phnum; ++i)
 	{
@@ -296,107 +291,7 @@ pax_setflags(PyObject *self, PyObject *args)
=20
 		if(phdr.p_type =3D=3D PT_PAX_FLAGS)
 		{
-			//PAGEEXEC
-			if(pax_flags & PF_PAGEEXEC)
-			{
-				phdr.p_flags |=3D PF_PAGEEXEC;
-				phdr.p_flags &=3D ~PF_NOPAGEEXEC;
-			}
-			if(pax_flags & PF_NOPAGEEXEC)
-			{
-				phdr.p_flags &=3D ~PF_PAGEEXEC;
-				phdr.p_flags |=3D PF_NOPAGEEXEC;
-			}
-			if((pax_flags & PF_PAGEEXEC) && (pax_flags & PF_NOPAGEEXEC))
-			{
-				phdr.p_flags &=3D ~PF_PAGEEXEC;
-				phdr.p_flags &=3D ~PF_NOPAGEEXEC;
-			}
-
-			//SEGMEXEC
-			if(pax_flags & PF_SEGMEXEC)
-			{
-				phdr.p_flags |=3D PF_SEGMEXEC;
-				phdr.p_flags &=3D ~PF_NOSEGMEXEC;
-			}
-			if(pax_flags & PF_NOSEGMEXEC)
-			{
-				phdr.p_flags &=3D ~PF_SEGMEXEC;
-				phdr.p_flags |=3D PF_NOSEGMEXEC;
-			}
-			if((pax_flags & PF_SEGMEXEC) && (pax_flags & PF_NOSEGMEXEC))
-			{
-				phdr.p_flags &=3D ~PF_SEGMEXEC;
-				phdr.p_flags &=3D ~PF_NOSEGMEXEC;
-			}
-
-			//MPROTECT
-			if(pax_flags & PF_MPROTECT)
-			{
-				phdr.p_flags |=3D PF_MPROTECT;
-				phdr.p_flags &=3D ~PF_NOMPROTECT;
-			}
-			if(pax_flags & PF_NOMPROTECT)
-			{
-				phdr.p_flags &=3D ~PF_MPROTECT;
-				phdr.p_flags |=3D PF_NOMPROTECT;
-			}
-			if((pax_flags & PF_MPROTECT) && (pax_flags & PF_NOMPROTECT))
-			{
-				phdr.p_flags &=3D ~PF_MPROTECT;
-				phdr.p_flags &=3D ~PF_NOMPROTECT;
-			}
-
-			//EMUTRAMP
-			if(pax_flags & PF_EMUTRAMP)
-			{
-				phdr.p_flags |=3D PF_EMUTRAMP;
-				phdr.p_flags &=3D ~PF_NOEMUTRAMP;
-			}
-			if(pax_flags & PF_NOEMUTRAMP)
-			{
-				phdr.p_flags &=3D ~PF_EMUTRAMP;
-				phdr.p_flags |=3D PF_NOEMUTRAMP;
-			}
-			if((pax_flags & PF_EMUTRAMP) && (pax_flags & PF_NOEMUTRAMP))
-			{
-				phdr.p_flags &=3D ~PF_EMUTRAMP;
-				phdr.p_flags &=3D ~PF_NOEMUTRAMP;
-			}
-
-			//RANDMMAP
-			if(pax_flags & PF_RANDMMAP)
-			{
-				phdr.p_flags |=3D PF_RANDMMAP;
-				phdr.p_flags &=3D ~PF_NORANDMMAP;
-			}
-			if(pax_flags & PF_NORANDMMAP)
-			{
-				phdr.p_flags &=3D ~PF_RANDMMAP;
-				phdr.p_flags |=3D PF_NORANDMMAP;
-			}
-			if((pax_flags & PF_RANDMMAP) && (pax_flags & PF_NORANDMMAP))
-			{
-				phdr.p_flags &=3D ~PF_RANDMMAP;
-				phdr.p_flags &=3D ~PF_NORANDMMAP;
-			}
-
-			//RANDEXEC
-			if(pax_flags & PF_RANDEXEC)
-			{
-				phdr.p_flags |=3D PF_RANDEXEC;
-				phdr.p_flags &=3D ~PF_NORANDEXEC;
-			}
-			if(pax_flags & PF_NORANDEXEC)
-			{
-				phdr.p_flags &=3D ~PF_RANDEXEC;
-				phdr.p_flags |=3D PF_NORANDEXEC;
-			}
-			if((pax_flags & PF_RANDEXEC) && (pax_flags & PF_NORANDEXEC))
-			{
-				phdr.p_flags &=3D ~PF_RANDEXEC;
-				phdr.p_flags &=3D ~PF_NORANDEXEC;
-			}
+			phdr.p_flags =3D pax_flags;
=20
 			if(!gelf_update_phdr(elf, i, &phdr))
 			{

diff --git a/scripts/revdep-pax b/scripts/revdep-pax
index 326007e..7a8da3e 100755
--- a/scripts/revdep-pax
+++ b/scripts/revdep-pax
@@ -8,7 +8,6 @@ import re
 import pax
=20
 def get_ldd_linkings(binary):
-
 	try:
 		#TODO: when subprocess.DEVNULL makes it to python, change this: http:/=
/bugs.python.org/issue5870
 		ldd_output =3D subprocess.check_output(['/usr/bin/ldd', binary], stder=
r=3Dsubprocess.PIPE)
@@ -83,8 +82,8 @@ def print_forward_linkings( forward_linkings, so2librar=
y_mappings, verbose ):
 	for binary in forward_linkings:
=20
 		try:
-			binary_flags =3D pax.getflags(binary)
-			sv =3D "%s ( %s )" % ( binary, binary_flags )
+			( binary_flags, binary_pax_flags ) =3D pax.getflags(binary)
+			sv =3D '%s ( %s )' % ( binary, binary_flags )
 			s =3D sv
 		except:
 			missing_binaries.append(binary)
@@ -94,10 +93,10 @@ def print_forward_linkings( forward_linkings, so2libr=
ary_mappings, verbose ):
 		for soname in forward_linkings[binary]:
 			try:
 				library =3D so2library_mappings[soname]
-				library_flags =3D pax.getflags(library)
-				sv =3D "%s\n\t%s\t%s ( %s )" % ( sv, soname, library, library_flags =
)
+				( library_flags, library_pax_flags ) =3D pax.getflags(library)
+				sv =3D '%s\n\t%s\t%s ( %s )' % ( sv, soname, library, library_flags =
)
 				if binary_flags !=3D library_flags:
-					s =3D "%s\n\t%s\t%s ( %s )" % ( s, soname, library, library_flags )
+					s =3D '%s\n\t%s\t%s ( %s )' % ( s, soname, library, library_flags )
 					count =3D count + 1
 			except:
 				missing_links.append(soname)
@@ -145,8 +144,8 @@ def print_reverse_linkings( reverse_linkings, so2libr=
ary_mappings, verbose ):
=20
 		try:
 			library =3D so2library_mappings[soname]
-			library_flags =3D pax.getflags(library)
-			sv =3D "%s\t%s ( %s )" % ( soname, library, library_flags )
+			( library_flags, library_pax_flags ) =3D pax.getflags(library)
+			sv =3D '%s\t%s ( %s )' % ( soname, library, library_flags )
 			s =3D sv
 		except:
 			missing_sonames.append(soname)
@@ -155,10 +154,10 @@ def print_reverse_linkings( reverse_linkings, so2li=
brary_mappings, verbose ):
 		count =3D 0
 		for binary in reverse_linkings[soname]:
 			try:
-				binary_flags =3D pax.getflags(binary)
-				sv =3D "%s\n\t%s ( %s )" % ( sv, binary, binary_flags )
+				( binary_flags, binary_pax_flags ) =3D pax.getflags(binary)
+				sv =3D '%s\n\t%s ( %s )' % ( sv, binary, binary_flags )
 				if library_flags !=3D binary_flags:
-					s =3D "%s\n\t%s ( %s )" % ( s, binary, binary_flags )
+					s =3D '%s\n\t%s ( %s )' % ( s, binary, binary_flags )
 					count =3D count + 1
 			except:
 				missing_links.append(binary)
@@ -203,12 +202,13 @@ def run_usage():
 	print 'Program Name : revdep-pax'
 	print 'Description  : Get or set pax flags on an ELF object'
 	print
-	print 'Usage        : revdep-pax -f [-v]         print out all forward =
mappings for all system binaries'
-	print '             : revdep-pax -r [-v]         print out all reverse =
mappints for all system sonames'
-	print '             : revdep-pax -b BINARY [-v]  print all forward mapp=
ings only for BINARY'
-	print '             : revdep-pax -s SONAME [-v]  print all reverse mapp=
ings only for SONAME'
-	print '             : revdep-pax [-h]            print out this help'
-	print '             : -v                         verbose, otherwise jus=
t print mismatched flags'
+	print 'Usage        : revdep-pax -f [-mv]         print out all forward=
 mappings for all system binaries'
+	print '             : revdep-pax -r [-mv]         print out all reverse=
 mappints for all system sonames'
+	print '             : revdep-pax -b BINARY [-mv]  print all forward map=
pings only for BINARY'
+	print '             : revdep-pax -s SONAME [-mv]  print all reverse map=
pings only for SONAME'
+	print '             : revdep-pax [-h]             print out this help'
+	print '             : -v                          verbose, otherwise ju=
st print mismatched flags'
+	print '             : -m                          prompt to mark the fo=
und objects'
 	print
=20
=20
@@ -225,28 +225,28 @@ def run_reverse(verbose):
 	print
=20
=20
-def run_binary(binary, verbose):
+def run_binary(binary, verbose, mark):
 	( linkings, mappings ) =3D get_ldd_linkings(binary)
-
-	binary_flags =3D pax.getflags(binary)
-	print binary, '(', binary_flags, ')'
+	( binary_flags, binary_pax_flags ) =3D pax.getflags(binary)
+	print '%s (%s)' % ( binary, binary_flags )
 	print
=20
-	count =3D 0
+	mismatched_libraries =3D []
+
 	for soname in linkings:
 		try:
 			library =3D mappings[soname]
-			library_flags =3D pax.getflags(library)
+			( library_flags, library_pax_flags ) =3D pax.getflags(library)
 			if verbose:
-				print '\t', soname, '\t', library, '(', library_flags, ')'
-			else:
-				if binary_flags !=3D library_flags:
-					print '\t', soname, '\t', library, '(', library_flags, ')'
-					count =3D count + 1
+				print '\t%s\t%s ( %s )' % ( soname, library, library_flags )
+			if binary_flags !=3D library_flags:
+				mismatched_libraries.append(library)
+				if not verbose:
+					print '\t%s\t%s ( %s )' % ( soname, library, library_flags )
 		except:
-			print "file for soname %s not found" % soname
+			print 'file for soname %s not found' % soname
=20
-	if count =3D=3D 0:
+	if len(mismatched_libraries) =3D=3D 0:
 		print
 		print '\tNo mismatches'
 		print
@@ -254,6 +254,27 @@ def run_binary(binary, verbose):
 		print
 		print '\tMismatches'
 		print
+		if mark:
+			print '\tWill mark libraries with %s' % binary_flags
+			print
+			for library in mismatched_libraries:
+				do_marking =3D False
+				while True:
+					ans =3D raw_input('\tSet flags for %s (y/n): ' % library)
+					if ans =3D=3D 'y':
+						do_marking =3D True
+						break
+					elif ans =3D=3D 'n':
+						do_marking =3D False
+						break
+					else:
+						print '\t\tPlease enter y or n'
+
+				if do_marking:
+					pax.setflags(library, binary_pax_flags)
+					( library_flags, library_pax_flags ) =3D pax.getflags(library)
+					print '\t%s ( %s )' % ( library, library_flags )
+					print
=20
=20
 def invert_so2library_mappings( so2library_mappings ):
@@ -263,7 +284,7 @@ def invert_so2library_mappings( so2library_mappings )=
:
 	return library2soname_mappings
=20
=20
-def run_soname(name, verbose, use_soname):
+def run_soname(name, verbose, use_soname, mark):
 	( forward_linkings, so2library_mappings ) =3D get_forward_linkings()
 	reverse_linkings =3D invert_linkings( forward_linkings )
=20
@@ -276,14 +297,14 @@ def run_soname(name, verbose, use_soname):
 	linkings =3D reverse_linkings[soname]
 	library =3D so2library_mappings[soname]
=20
-	library_flags =3D pax.getflags(library)
+	( library_flags, library_pax_flags ) =3D pax.getflags(library)
 	print soname, '\t', library, '(', library_flags, ')'
 	print
=20
 	count =3D 0
 	for binary in linkings:
 		try:
-			binary_flags =3D pax.getflags(binary)
+			( binary_flags, binary_pax_flags ) =3D pax.getflags(binary)
 			if verbose:
 				print '\t', binary, '(', binary_flags, ')'
 			else:
@@ -291,7 +312,7 @@ def run_soname(name, verbose, use_soname):
 					print '\t', binary, '(', binary_flags, ')'
 					count =3D count + 1
 		except:
-			print "cannot obtain pax flags for %s" % binary
+			print 'cannot obtain pax flags for %s' % binary
=20
 	if count =3D=3D 0:
 		print
@@ -305,7 +326,7 @@ def run_soname(name, verbose, use_soname):
=20
 def main():
 	try:
-		opts, args =3D getopt.getopt(sys.argv[1:], 'hfrb:s:l:v')
+		opts, args =3D getopt.getopt(sys.argv[1:], 'hfrb:s:l:vm')
 	except getopt.GetoptError, err:
 		print str(err) # will print something like 'option -a not recognized'
 		run_usage()
@@ -324,6 +345,7 @@ def main():
 	library =3D None
=20
 	verbose =3D False
+	mark =3D False
=20
 	opt_count =3D 0
=20
@@ -348,6 +370,8 @@ def main():
 			opt_count +=3D 1
 		elif o =3D=3D '-v':
 			verbose =3D True
+		elif o =3D=3D '-m':
+			mark =3D True
 		else:
 			print 'Option included in getopt but not handled here!'
 			print 'Please file a bug'
@@ -361,12 +385,12 @@ def main():
 	elif do_reverse:
 		run_reverse(verbose)
 	elif binary !=3D None:
-		run_binary(binary, verbose)
+		run_binary(binary, verbose, mark)
 	elif soname !=3D None:
-		run_soname(soname, verbose, True)
+		run_soname(soname, verbose, True, mark)
 	elif library !=3D None:
 		library =3D os.path.realpath(library)
-		run_soname(library, verbose, False)
+		run_soname(library, verbose, False, mark)
=20
 if __name__ =3D=3D '__main__':
     main()