diff options
author | Thialfihar <thialfihar@gmail.com> | 2010-04-23 17:00:52 +0000 |
---|---|---|
committer | Thialfihar <thialfihar@gmail.com> | 2010-04-23 17:00:52 +0000 |
commit | 0e14dcb290d5b3c7acae1da5a64e611e293c7f06 (patch) | |
tree | a896c8e78fa6d4d83dd7b6f2ab04fcedd9e567a2 | |
parent | ab6c884bdf8329eee182fed929078135077afd1e (diff) | |
download | open-keychain-0e14dcb290d5b3c7acae1da5a64e611e293c7f06.tar.gz open-keychain-0e14dcb290d5b3c7acae1da5a64e611e293c7f06.tar.bz2 open-keychain-0e14dcb290d5b3c7acae1da5a64e611e293c7f06.zip |
layout adjustments, force portrait layout for EncryptFileActivity for now, (En|De)cryptFileActivity can now handle symmetric algorithms
-rw-r--r-- | AndroidManifest.xml | 1 | ||||
-rw-r--r-- | res/layout/encrypt_file.xml | 327 | ||||
-rw-r--r-- | res/values/strings.xml | 4 | ||||
-rw-r--r-- | src/org/thialfihar/android/apg/Apg.java | 142 | ||||
-rw-r--r-- | src/org/thialfihar/android/apg/AskForSecretKeyPassPhrase.java | 58 | ||||
-rw-r--r-- | src/org/thialfihar/android/apg/DecryptFileActivity.java | 14 | ||||
-rw-r--r-- | src/org/thialfihar/android/apg/DecryptMessageActivity.java | 2 | ||||
-rw-r--r-- | src/org/thialfihar/android/apg/EncryptFileActivity.java | 2 | ||||
-rw-r--r-- | src/org/thialfihar/android/apg/MainActivity.java | 2 |
9 files changed, 317 insertions, 235 deletions
diff --git a/AndroidManifest.xml b/AndroidManifest.xml index b9dd47fd0..1475080c2 100644 --- a/AndroidManifest.xml +++ b/AndroidManifest.xml @@ -74,6 +74,7 @@ <activity android:name=".EncryptFileActivity" android:label="@string/title_encryptFile" + android:screenOrientation="portrait" android:configChanges="keyboardHidden|orientation|keyboard"> <intent-filter> diff --git a/res/layout/encrypt_file.xml b/res/layout/encrypt_file.xml index 0c388fbe7..2945cf89f 100644 --- a/res/layout/encrypt_file.xml +++ b/res/layout/encrypt_file.xml @@ -21,204 +21,213 @@ android:orientation="vertical" android:paddingTop="5dip"> - <LinearLayout - android:layout_width="fill_parent" - android:layout_height="wrap_content" - android:orientation="horizontal" - android:paddingLeft="5dip" - android:paddingRight="5dip"> - - <TextView - android:id="@+id/label_filename" - android:textAppearance="?android:attr/textAppearanceMedium" - android:layout_width="50dip" - android:layout_height="wrap_content" - android:text="@string/label_file"/> - - <EditText - android:id="@+id/filename" - android:layout_width="0dip" - android:layout_height="wrap_content" - android:layout_weight="1"/> - - <ImageButton - android:id="@+id/btn_browse" - android:src="@drawable/ic_launcher_folder_small" - android:layout_height="wrap_content" - android:layout_width="wrap_content"/> - - </LinearLayout> + <LinearLayout + android:layout_width="fill_parent" + android:layout_height="wrap_content" + android:orientation="horizontal" + android:paddingLeft="5dip"> - <CheckBox - android:id="@+id/ascii_armour" - android:text="@string/ascii_armour" - android:layout_height="wrap_content" - android:layout_width="wrap_content"/> + <TextView + android:id="@+id/label_filename" + android:textAppearance="?android:attr/textAppearanceMedium" + android:layout_width="50dip" + android:layout_height="wrap_content" + android:text="@string/label_file"/> - <TabHost - android:id="@+id/tab_host" - android:layout_weight="1" - android:layout_width="fill_parent" - android:layout_height="0dip"> + <EditText + android:id="@+id/filename" + android:layout_width="0dip" + android:layout_height="wrap_content" + android:layout_weight="1"/> - <TabWidget - android:id="@android:id/tabs" - android:layout_width="fill_parent" - android:layout_height="65dip"/> + <ImageButton + android:id="@+id/btn_browse" + android:src="@drawable/ic_launcher_folder_small" + android:layout_height="wrap_content" + android:layout_width="wrap_content"/> - <FrameLayout - android:id="@android:id/tabcontent" - android:layout_width="fill_parent" - android:layout_height="fill_parent" - android:paddingTop="65dip"> + </LinearLayout> -<!-- --> - <LinearLayout - android:id="@+id/tab_asymmetric" - android:layout_width="fill_parent" + <TableLayout android:layout_height="wrap_content" - android:orientation="vertical"> + android:layout_width="fill_parent" + android:stretchColumns="1" + android:paddingLeft="6dip"> - <LinearLayout - android:layout_width="fill_parent" - android:layout_height="wrap_content" - android:paddingBottom="3dip" - android:orientation="horizontal"> + <TableRow> - <CheckBox - android:text="@string/sign" - android:id="@+id/sign" - android:layout_width="wrap_content" - android:layout_height="wrap_content" - android:layout_gravity="center_vertical"/> + <TextView android:id="@+id/label_algorithm" + android:text="Algorithm:" + android:layout_height="wrap_content" + android:layout_width="wrap_content" + android:layout_gravity="center_vertical" + android:paddingRight="10dip"/> - <LinearLayout - android:orientation="vertical" - android:layout_height="wrap_content" + <Spinner + android:id="@+id/algorithm" android:layout_width="fill_parent" - android:layout_gravity="center_vertical" - android:paddingRight="5dip"> - - <TextView - android:id="@+id/main_user_id" - android:text="Main User Id" - android:textAppearance="?android:attr/textAppearanceMedium" - android:layout_width="wrap_content" - android:layout_height="wrap_content" - android:layout_gravity="right"/> + android:layout_height="wrap_content"/> - <TextView - android:id="@+id/main_user_id_rest" - android:text="Main User Id Rest" - android:textAppearance="?android:attr/textAppearanceSmall" - android:layout_width="wrap_content" - android:layout_height="wrap_content" - android:layout_gravity="right"/> + </TableRow> - </LinearLayout> + <TableRow> - </LinearLayout> + <TextView + android:id="@+id/label_ascii_armour" + android:text="@string/ascii_armour" + android:layout_gravity="center_vertical" + android:paddingRight="10dip" + android:layout_height="wrap_content" + android:layout_width="wrap_content"/> - <View - android:id="@+id/separator" - android:layout_width="fill_parent" - android:layout_height="1dip" - android:background="?android:attr/listDivider"/> + <CheckBox + android:id="@+id/ascii_armour" + android:layout_height="wrap_content" + android:layout_width="wrap_content" + android:layout_gravity="right|center_vertical"/> - <TextView - android:id="@+id/label_encrypt_keys" - android:text="@string/label_encrypt_keys" - android:textAppearance="?android:attr/textAppearanceMedium" - android:layout_width="wrap_content" - android:layout_height="wrap_content"/> + </TableRow> - <ListView - android:id="@+id/public_key_list" - android:choiceMode="multipleChoice" - android:layout_width="fill_parent" - android:layout_height="0dip" - android:layout_weight="1"/> + </TableLayout> - </LinearLayout> - <!-- --> - <ScrollView - android:id="@+id/tab_symmetric" + <TabHost + android:id="@+id/tab_host" + android:layout_weight="1" android:layout_width="fill_parent" - android:layout_height="fill_parent" - android:fillViewport="true"> + android:layout_height="0dip"> - <LinearLayout + <TabWidget + android:id="@android:id/tabs" + android:layout_width="fill_parent" + android:layout_height="65dip"/> + + <FrameLayout + android:id="@android:id/tabcontent" android:layout_width="fill_parent" android:layout_height="fill_parent" - android:orientation="vertical" - android:paddingTop="3dip"> + android:paddingTop="65dip"> - <TableLayout - android:layout_height="fill_parent" + <!-- --> + <LinearLayout + android:id="@+id/tab_asymmetric" android:layout_width="fill_parent" - android:stretchColumns="1" - android:layout_marginRight="?android:attr/scrollbarSize" - android:paddingLeft="6dip"> - - <TableRow - android:layout_marginBottom="5dip"> - - <TextView android:id="@+id/label_algorithm" - android:text="Algorithm:" - android:layout_height="wrap_content" - android:layout_width="wrap_content" - android:layout_gravity="center_vertical" - android:paddingRight="10dip"/> - - <Spinner - android:id="@+id/algorithm" + android:layout_height="fill_parent" + android:orientation="vertical"> + + <LinearLayout + android:layout_width="fill_parent" + android:layout_height="wrap_content" + android:paddingBottom="3dip" + android:orientation="horizontal"> + + <CheckBox + android:text="@string/sign" + android:id="@+id/sign" + android:layout_width="wrap_content" + android:layout_height="wrap_content" + android:layout_gravity="center_vertical"/> + + <LinearLayout + android:orientation="vertical" + android:layout_height="wrap_content" android:layout_width="fill_parent" - android:layout_height="wrap_content"/> + android:layout_gravity="center_vertical" + android:paddingRight="5dip"> + + <TextView + android:id="@+id/main_user_id" + android:text="Main User Id" + android:textAppearance="?android:attr/textAppearanceMedium" + android:layout_width="wrap_content" + android:layout_height="wrap_content" + android:layout_gravity="right"/> + + <TextView + android:id="@+id/main_user_id_rest" + android:text="Main User Id Rest" + android:textAppearance="?android:attr/textAppearanceSmall" + android:layout_width="wrap_content" + android:layout_height="wrap_content" + android:layout_gravity="right"/> + + </LinearLayout> + + </LinearLayout> + + <View + android:id="@+id/separator" + android:layout_width="fill_parent" + android:layout_height="1dip" + android:background="?android:attr/listDivider"/> - </TableRow> + <TextView + android:id="@+id/label_encrypt_keys" + android:text="@string/label_encrypt_keys" + android:textAppearance="?android:attr/textAppearanceMedium" + android:layout_width="wrap_content" + android:layout_height="wrap_content"/> - <TableRow> + <ListView + android:id="@+id/public_key_list" + android:choiceMode="multipleChoice" + android:layout_width="fill_parent" + android:layout_height="fill_parent"/> - <TextView android:id="@+id/label_pass_phrase" - android:text="Pass phrase" - android:layout_height="wrap_content" - android:layout_width="wrap_content" - android:layout_gravity="center_vertical" - android:paddingRight="10dip"/> + </LinearLayout> + <!-- --> + <LinearLayout + android:id="@+id/tab_symmetric" + android:layout_width="fill_parent" + android:layout_height="fill_parent" + android:orientation="vertical" + android:paddingTop="3dip"> + + <TableLayout + android:layout_height="fill_parent" + android:layout_width="fill_parent" + android:stretchColumns="1" + android:paddingLeft="6dip"> - <EditText android:id="@+id/pass_phrase" - android:layout_height="wrap_content" - android:layout_width="fill_parent" - android:inputType="textPassword"/> + <TableRow> - </TableRow> + <TextView android:id="@+id/label_pass_phrase" + android:text="Pass phrase" + android:layout_height="wrap_content" + android:layout_width="wrap_content" + android:layout_gravity="center_vertical" + android:paddingRight="10dip"/> - <TableRow> + <EditText android:id="@+id/pass_phrase" + android:layout_height="wrap_content" + android:layout_width="fill_parent" + android:inputType="textPassword"/> - <TextView android:id="@+id/label_pass_phrase_again" - android:text="Again:" - android:layout_height="wrap_content" - android:layout_width="wrap_content" - android:layout_gravity="center_vertical" - android:paddingRight="10dip"/> + </TableRow> - <EditText android:id="@+id/pass_phrase_again" - android:layout_height="wrap_content" - android:layout_width="fill_parent" - android:inputType="textPassword"/> + <TableRow> - </TableRow> + <TextView android:id="@+id/label_pass_phrase_again" + android:text="Again:" + android:layout_height="wrap_content" + android:layout_width="wrap_content" + android:layout_gravity="center_vertical" + android:paddingRight="10dip"/> - </TableLayout> + <EditText android:id="@+id/pass_phrase_again" + android:layout_height="wrap_content" + android:layout_width="fill_parent" + android:inputType="textPassword"/> + </TableRow> - </LinearLayout> + </TableLayout> - </ScrollView> - </FrameLayout> + </LinearLayout> + + </FrameLayout> - </TabHost> + </TabHost> <LinearLayout android:orientation="horizontal" diff --git a/res/values/strings.xml b/res/values/strings.xml index 5ad12d0ff..02a93d5bd 100644 --- a/res/values/strings.xml +++ b/res/values/strings.xml @@ -45,8 +45,8 @@ <string name="btn_save">Save</string> <string name="btn_doNotSave">Cancel</string> - <string name="tab_symmetric">Symmetric</string> - <string name="tab_asymmetric">Asymmetric</string> + <string name="tab_symmetric">Use Pass Phrase</string> + <string name="tab_asymmetric">Use Public Key</string> <string name="menu_about">About</string> <string name="menu_addAccount">Add GMail Account</string> diff --git a/src/org/thialfihar/android/apg/Apg.java b/src/org/thialfihar/android/apg/Apg.java index a5daca0cb..79e0a8758 100644 --- a/src/org/thialfihar/android/apg/Apg.java +++ b/src/org/thialfihar/android/apg/Apg.java @@ -41,6 +41,7 @@ import java.util.Comparator; import java.util.Date;
import java.util.GregorianCalendar;
import java.util.HashMap;
+import java.util.Iterator;
import java.util.Vector;
import java.util.regex.Pattern;
@@ -66,6 +67,7 @@ import org.bouncycastle2.openpgp.PGPLiteralDataGenerator; import org.bouncycastle2.openpgp.PGPObjectFactory;
import org.bouncycastle2.openpgp.PGPOnePassSignature;
import org.bouncycastle2.openpgp.PGPOnePassSignatureList;
+import org.bouncycastle2.openpgp.PGPPBEEncryptedData;
import org.bouncycastle2.openpgp.PGPPrivateKey;
import org.bouncycastle2.openpgp.PGPPublicKey;
import org.bouncycastle2.openpgp.PGPPublicKeyEncryptedData;
@@ -1361,24 +1363,60 @@ public class Apg { throw new GeneralException("data not valid encryption data");
}
+ // TODO: currently we always only look at the first known key
// find the secret key
PGPSecretKey secretKey = null;
- for (PGPPublicKeyEncryptedData pbe :
- new IterableIterator<PGPPublicKeyEncryptedData>(enc.getEncryptedDataObjects())) {
- secretKey = findSecretKey(pbe.getKeyID());
- if (secretKey != null) {
- break;
+ Iterator it = enc.getEncryptedDataObjects();
+ while (it.hasNext()) {
+ Object obj = it.next();
+ if (obj instanceof PGPPublicKeyEncryptedData) {
+ PGPPublicKeyEncryptedData pbe = (PGPPublicKeyEncryptedData) obj;
+ secretKey = findSecretKey(pbe.getKeyID());
+ if (secretKey != null) {
+ break;
+ }
}
}
+
if (secretKey == null) {
- throw new GeneralException("couldn't find a secret key to decrypt");
+ return 0;
}
return secretKey.getKeyID();
}
+ public static boolean hasSymmetricEncryption(InputStream inStream)
+ throws GeneralException, IOException {
+ InputStream in = PGPUtil.getDecoderStream(inStream);
+ PGPObjectFactory pgpF = new PGPObjectFactory(in);
+ PGPEncryptedDataList enc;
+ Object o = pgpF.nextObject();
+
+ // the first object might be a PGP marker packet.
+ if (o instanceof PGPEncryptedDataList) {
+ enc = (PGPEncryptedDataList) o;
+ } else {
+ enc = (PGPEncryptedDataList) pgpF.nextObject();
+ }
+
+ if (enc == null) {
+ throw new GeneralException("data not valid encryption data");
+ }
+
+ Iterator it = enc.getEncryptedDataObjects();
+ while (it.hasNext()) {
+ Object obj = it.next();
+ if (obj instanceof PGPPBEEncryptedData) {
+ return true;
+ }
+ }
+
+ return false;
+ }
+
public static Bundle decrypt(InputStream inStream, OutputStream outStream,
- String passPhrase, ProgressDialogUpdater progress)
+ String passPhrase, ProgressDialogUpdater progress,
+ boolean assumeSymmetric)
throws IOException, GeneralException, PGPException, SignatureException {
Bundle returnData = new Bundle();
InputStream in = PGPUtil.getDecoderStream(inStream);
@@ -1399,32 +1437,66 @@ public class Apg { throw new GeneralException("data not valid encryption data");
}
- progress.setProgress("finding key...", 10, 100);
- // find the secret key
- PGPPublicKeyEncryptedData pbe = null;
- PGPSecretKey secretKey = null;
- for (PGPPublicKeyEncryptedData encData :
- new IterableIterator<PGPPublicKeyEncryptedData>(enc.getEncryptedDataObjects())) {
- secretKey = findSecretKey(encData.getKeyID());
- if (secretKey != null) {
- pbe = encData;
- break;
+ InputStream clear = null;
+ PGPEncryptedData encryptedData = null;
+
+ // TODO: currently we always only look at the first known key or symmetric encryption,
+ // there might be more...
+ if (assumeSymmetric) {
+ PGPPBEEncryptedData pbe = null;
+ Iterator it = enc.getEncryptedDataObjects();
+ // find secret key
+ while (it.hasNext()) {
+ Object obj = it.next();
+ if (obj instanceof PGPPBEEncryptedData) {
+ pbe = (PGPPBEEncryptedData) obj;
+ break;
+ }
}
- }
- if (secretKey == null) {
- throw new GeneralException("couldn't find a secret key to decrypt");
- }
- progress.setProgress("extracting key...", 20, 100);
- PGPPrivateKey privateKey = null;
- try {
- privateKey = secretKey.extractPrivateKey(passPhrase.toCharArray(),
- new BouncyCastleProvider());
- } catch (PGPException e) {
- throw new PGPException("wrong pass phrase");
+ if (pbe == null) {
+ throw new GeneralException("couldn't find a packet with symmetric encryption");
+ }
+
+ progress.setProgress("decrypting data...", 20, 100);
+ clear = pbe.getDataStream(passPhrase.toCharArray(), new BouncyCastleProvider());
+ encryptedData = pbe;
+ } else {
+ progress.setProgress("finding key...", 10, 100);
+ PGPPublicKeyEncryptedData pbe = null;
+ PGPSecretKey secretKey = null;
+ Iterator it = enc.getEncryptedDataObjects();
+ // find secret key
+ while (it.hasNext()) {
+ Object obj = it.next();
+ if (obj instanceof PGPPublicKeyEncryptedData) {
+ PGPPublicKeyEncryptedData encData = (PGPPublicKeyEncryptedData) obj;
+ secretKey = findSecretKey(encData.getKeyID());
+ if (secretKey != null) {
+ pbe = encData;
+ break;
+ }
+ }
+ }
+
+ if (secretKey == null) {
+ throw new GeneralException("couldn't find a secret key to decrypt");
+ }
+
+ progress.setProgress("extracting key...", 20, 100);
+ PGPPrivateKey privateKey = null;
+ try {
+ privateKey = secretKey.extractPrivateKey(passPhrase.toCharArray(),
+ new BouncyCastleProvider());
+ } catch (PGPException e) {
+ throw new PGPException("wrong pass phrase");
+ }
+
+ progress.setProgress("decrypting data...", 30, 100);
+ clear = pbe.getDataStream(privateKey, new BouncyCastleProvider());
+ encryptedData = pbe;
}
- progress.setProgress("decrypting data...", 30, 100);
- InputStream clear = pbe.getDataStream(privateKey, new BouncyCastleProvider());
+
PGPObjectFactory plainFact = new PGPObjectFactory(clear);
Object dataChunk = plainFact.nextObject();
PGPOnePassSignature signature = null;
@@ -1511,15 +1583,15 @@ public class Apg { }
// TODO: add integrity somewhere
- if (pbe.isIntegrityProtected()) {
+ if (encryptedData.isIntegrityProtected()) {
progress.setProgress("verifying integrity...", 90, 100);
- if (!pbe.verify()) {
- System.err.println("message failed integrity check");
+ if (encryptedData.verify()) {
+ // passed
} else {
- System.err.println("message integrity check passed");
+ // failed
}
} else {
- System.err.println("no message integrity check");
+ // no integrity check
}
progress.setProgress("done.", 100, 100);
diff --git a/src/org/thialfihar/android/apg/AskForSecretKeyPassPhrase.java b/src/org/thialfihar/android/apg/AskForSecretKeyPassPhrase.java index 06e9c6d0a..a6fe557c4 100644 --- a/src/org/thialfihar/android/apg/AskForSecretKeyPassPhrase.java +++ b/src/org/thialfihar/android/apg/AskForSecretKeyPassPhrase.java @@ -26,9 +26,6 @@ import android.app.Dialog; import android.content.DialogInterface;
import android.text.InputType;
import android.text.method.PasswordTransformationMethod;
-import android.view.KeyEvent;
-import android.view.View;
-import android.view.View.OnKeyListener;
import android.view.ViewGroup.LayoutParams;
import android.widget.EditText;
import android.widget.LinearLayout;
@@ -43,36 +40,25 @@ public class AskForSecretKeyPassPhrase { PassPhraseCallbackInterface callback) {
AlertDialog.Builder alert = new AlertDialog.Builder(context);
- final PGPSecretKey secretKey =
- Apg.getMasterKey(Apg.findSecretKeyRing(secretKeyId));
- if (secretKey == null) {
- return null;
- }
+ alert.setTitle(R.string.title_authentification);
- String userId = Apg.getMainUserIdSafe(context, secretKey);
+ final PGPSecretKey secretKey;
- alert.setTitle(R.string.title_authentification);
- alert.setMessage("Pass phrase for " + userId);
+ if (secretKeyId == 0) {
+ secretKey = null;
+ alert.setMessage("Pass phrase");
+ } else {
+ secretKey = Apg.getMasterKey(Apg.findSecretKeyRing(secretKeyId));
+ if (secretKey == null) {
+ return null;
+ }
+ String userId = Apg.getMainUserIdSafe(context, secretKey);
+ alert.setMessage("Pass phrase for " + userId);
+ }
final EditText input = new EditText(context);
input.setInputType(InputType.TYPE_TEXT_VARIATION_PASSWORD);
input.setTransformationMethod(new PasswordTransformationMethod());
- input.setOnKeyListener(new OnKeyListener() {
- public boolean onKey(View v, int keyCode, KeyEvent event) {
- // If the event is a key-down event on the "enter" button
- if (event.getAction() == KeyEvent.ACTION_DOWN &&
- keyCode == KeyEvent.KEYCODE_ENTER) {
- try {
- ((AlertDialog) v.getParent()).getButton(AlertDialog.BUTTON_POSITIVE)
- .performClick();
- } catch (ClassCastException e) {
- // don't do anything if we're not in that dialog
- }
- return true;
- }
- return false;
- }
- });
// 5dip padding
int padding = (int) (10 * context.getResources().getDisplayMetrics().densityDpi / 160);
LinearLayout layout = new LinearLayout(context);
@@ -91,14 +77,16 @@ public class AskForSecretKeyPassPhrase { public void onClick(DialogInterface dialog, int id) {
activity.removeDialog(Id.dialog.pass_phrase);
String passPhrase = "" + input.getText();
- try {
- secretKey.extractPrivateKey(passPhrase.toCharArray(),
- new BouncyCastleProvider());
- } catch (PGPException e) {
- Toast.makeText(activity,
- R.string.wrong_pass_phrase,
- Toast.LENGTH_SHORT).show();
- return;
+ if (secretKey != null) {
+ try {
+ secretKey.extractPrivateKey(passPhrase.toCharArray(),
+ new BouncyCastleProvider());
+ } catch (PGPException e) {
+ Toast.makeText(activity,
+ R.string.wrong_pass_phrase,
+ Toast.LENGTH_SHORT).show();
+ return;
+ }
}
cb.passPhraseCallback(passPhrase);
}
diff --git a/src/org/thialfihar/android/apg/DecryptFileActivity.java b/src/org/thialfihar/android/apg/DecryptFileActivity.java index 47be98344..8e21eae11 100644 --- a/src/org/thialfihar/android/apg/DecryptFileActivity.java +++ b/src/org/thialfihar/android/apg/DecryptFileActivity.java @@ -59,6 +59,8 @@ public class DecryptFileActivity extends BaseActivity { private String mInputFilename = null;
private String mOutputFilename = null;
+ private boolean mAssumeSymmetricEncryption = false;
+
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
@@ -128,6 +130,16 @@ public class DecryptFileActivity extends BaseActivity { try {
InputStream in = new FileInputStream(mInputFilename);
setSecretKeyId(Apg.getDecryptionKeyId(in));
+ if (getSecretKeyId() == 0) {
+ // reopen the file to check whether there's symmetric encryption data in there
+ in = new FileInputStream(mInputFilename);
+ if (!Apg.hasSymmetricEncryption(in)) {
+ throw new Apg.GeneralException("no suitable keys found");
+ }
+ mAssumeSymmetricEncryption = true;
+ } else {
+ mAssumeSymmetricEncryption = false;
+ }
showDialog(Id.dialog.pass_phrase);
} catch (FileNotFoundException e) {
error = "file not found: " + e.getLocalizedMessage();
@@ -168,7 +180,7 @@ public class DecryptFileActivity extends BaseActivity { InputStream in = new FileInputStream(mInputFilename);
ByteArrayOutputStream out = new ByteArrayOutputStream();
- data = Apg.decrypt(in, out, Apg.getPassPhrase(), this);
+ data = Apg.decrypt(in, out, Apg.getPassPhrase(), this, mAssumeSymmetricEncryption);
out.close();
OutputStream fileOut = new FileOutputStream(mOutputFilename);
diff --git a/src/org/thialfihar/android/apg/DecryptMessageActivity.java b/src/org/thialfihar/android/apg/DecryptMessageActivity.java index 3f2a74987..d3a84eff5 100644 --- a/src/org/thialfihar/android/apg/DecryptMessageActivity.java +++ b/src/org/thialfihar/android/apg/DecryptMessageActivity.java @@ -204,7 +204,7 @@ public class DecryptMessageActivity extends BaseActivity { if (mSignedOnly) { data = Apg.verifyText(in, out, this); } else { - data = Apg.decrypt(in, out, Apg.getPassPhrase(), this); + data = Apg.decrypt(in, out, Apg.getPassPhrase(), this, false); } out.close(); diff --git a/src/org/thialfihar/android/apg/EncryptFileActivity.java b/src/org/thialfihar/android/apg/EncryptFileActivity.java index ce2cf9031..3f7db2b72 100644 --- a/src/org/thialfihar/android/apg/EncryptFileActivity.java +++ b/src/org/thialfihar/android/apg/EncryptFileActivity.java @@ -101,7 +101,7 @@ public class EncryptFileActivity extends BaseActivity { TabSpec ts2 = mTabHost.newTabSpec(TAB_SYMMETRIC);
ts2.setIndicator(getString(R.string.tab_symmetric),
- getResources().getDrawable(R.drawable.encrypted));
+ getResources().getDrawable(R.drawable.key));
ts2.setContent(R.id.tab_symmetric);
mTabHost.addTab(ts2);
diff --git a/src/org/thialfihar/android/apg/MainActivity.java b/src/org/thialfihar/android/apg/MainActivity.java index d6b04ce92..5562d6a7a 100644 --- a/src/org/thialfihar/android/apg/MainActivity.java +++ b/src/org/thialfihar/android/apg/MainActivity.java @@ -224,7 +224,7 @@ public class MainActivity extends BaseActivity { new SpannableString("Read the warnings!\n\n" + "Changes:\n" + " * OI File Manager support\n" + - " * file encryption\n" + + " * file encryption/decryption\n" + "\n" + "WARNING: be careful editing your existing keys, as they " + "WILL be stripped of certificates right now.\n" + |